Validate that a package doesn't depend on itself.

utils/tests/pub/validator_test.dart

Index: utils/tests/pub/validator_test.dart
diff --git a/utils/tests/pub/validator_test.dart b/utils/tests/pub/validator_test.dart
index 6f5764a95a4beabf134f6f640d83c019a743f784..32a833ba2a063f77de1a02cf47ceb5f017870ddc 100644
--- a/utils/tests/pub/validator_test.dart
+++ b/utils/tests/pub/validator_test.dart
@@ -531,5 +531,17 @@ main() {
         });
       });
     });
+
+    test('has a hosted dependency on itself', () {
+      dir(appPath, [
+        libPubspec("test_pkg", "1.0.0", [
+          {'hosted': 'foo'}

[Bob Nystrom, 2013/01/03 20:52:45]

Shouldn't this be "test_pkg"?

[nweiz, 2013/01/03 21:08:12]

Yes. The fact that this didn't have a version constraint caused a different
error, hence the false positive of this test. I've added a version constraint as
well.

[Bob Nystrom, 2013/01/03 21:56:17]

That makes me wonder if the expect should be more precise here so you can ensure
you get the validation warning you actually want.

[nweiz, 2013/01/03 22:18:22]

I'm not a big fan of searching for strings within error messages. I think having
validator-level granularity will be good enough most of the time.

+        ])
+      ]).scheduleCreate();
+
+      expectValidationWarning(dependency);
+
+      run();
+    });
   });
 }