config_service: Changed file links to point to latest revision instead of master

appengine/config_service/ui/test/config-ui/config-set_test.html

Index: appengine/config_service/ui/test/config-ui/config-set_test.html
diff --git a/appengine/config_service/ui/test/config-ui/config-set_test.html b/appengine/config_service/ui/test/config-ui/config-set_test.html
index 6259abdff2eb82b5f6addea76b55366d98a25dc8..7f25fedccbc86d018e99cb0d3d0bcbbca15e8329 100644
--- a/appengine/config_service/ui/test/config-ui/config-set_test.html
+++ b/appengine/config_service/ui/test/config-ui/config-set_test.html
@@ -50,7 +50,12 @@
             config_sets: [{
               files: [{path: "OWNERS"}, {path: "README.md"}],
               location: "https://test.com/valid-project-with-config-files",
-              last_import_attempt: {success: true}
+              last_import_attempt: {
+                success: true,
+                revision: {
+                  url: "https://test.com/valid-project-with-config-files"

[Sergey Berezin, 2017/07/27 21:14:14]

Make this URL different from `location`, so you can test which one gets picked.
Also, please add tests for each branch of the if-condition (when revision
exists, and when it doesn't), for better coverage.

Here you may benefit from a helper function to set up a server with a
customizable response, e.g. see
https://codereview.chromium.org/2990713002/diff/140001/appengine/config_servi...

[cwpayton, 2017/07/27 22:56:04]

Done.

+                }
+              }
             }]
           };
           server.respondWith(
@@ -91,7 +96,7 @@
             assert.equal(config_set.isLoading, false);
             assert.equal(config_set.files.length, 2);
             assert.equal(config_set.lastImportAttempt.success, true);
-            assert.equal(config_set.location, 
+            assert.equal(config_set.url,
               "https://test.com/valid-project-with-config-files");
             done();
           }.bind(config_set));
@@ -153,7 +158,7 @@
             assert.equal(config_set.isLoading, false);
             assert.equal(config_set.files.length, 0);
             assert.equal(config_set.lastImportAttempt.success, false);
-            assert.equal(config_set.location, 
+            assert.equal(config_set.url,
               "https://test.com/invalid-project-without-config-files");
             done();
           }.bind(config_set));