[GitHub] dewrich commented on a change in pull request #2191: revert changes that broke perl unit tests

[GitBox <gi...@apache.org>]

dewrich commented on a change in pull request #2191: revert changes that broke perl unit tests
URL: https://github.com/apache/incubator-trafficcontrol/pull/2191#discussion_r185232797
 
 

 ##########
 File path: traffic_ops/app/lib/TrafficOpsRoutes.pm
 ##########
 @@ -449,6 +449,13 @@ sub api_routes {
 	# -- CDNS: ROUTING
 	$r->get("/api/$version/cdns/routing")->over( authenticated => 1, not_ldap => 1 )->to( 'Cdn#routing', namespace => $namespace );
 
+	# -- CDNS: SNAPSHOT
+	$r->get("/api/$version/cdns/:name/snapshot")->over( authenticated => 1, not_ldap => 1 )->to( 'Topology#get_snapshot', namespace => $namespace );
+	$r->get("/api/$version/cdns/:name/snapshot/new")->over( authenticated => 1, not_ldap => 1 )->to( 'Topology#get_new_snapshot', namespace => $namespace );
+	$r->put( "/api/$version/cdns/:id/snapshot" => [ id => qr/\d+/ ] )->over( authenticated => 1, not_ldap => 1 )
+	->to( 'Topology#SnapshotCRConfig', namespace => $namespace );
+	$r->put("/api/$version/snapshot/:cdn_name")->over( authenticated => 1, not_ldap => 1 )->to( 'Topology#SnapshotCRConfig', namespace => $namespace );
 
 Review comment:
   @rob05c 
   ```Test Summary Report
   -------------------
   t/api/1.2/cdn.t                               (Wstat: 768 Tests: 59 Failed: 3)
     Failed tests:  13, 16, 18
     Non-zero exit status: 3
   t/api/1.2/config_files_apache_traffic_server.t (Wstat: 256 Tests: 54 Failed: 1)
     Failed test:  50
     Non-zero exit status: 1
   t/api/1.2/deliveryservice.t                   (Wstat: 768 Tests: 378 Failed: 3)
     Failed tests:  119, 245, 371
     Non-zero exit status: 3```

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services