Skip to content
This repository was archived by the owner on Apr 4, 2025. It is now read-only.

Commit 05f93f7

Browse files
committed
refactor: move e2e test files where protractor expects to find them
1 parent 13107a5 commit 05f93f7

18 files changed

+29
-17
lines changed
File renamed without changes.

examples/broken-example.html renamed to e2e/_unused/broken-example.html

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@ <h1> Angular Best Practices </h1>
5757
</li>
5858
</ul>
5959
</div>
60-
<script src="../node_modules/angular/angular.js"></script>
60+
<script src="../../node_modules/angular/angular.js"></script>
6161
<script>
6262
angular.module('bpAppModule', []);
6363
/**

examples/correct-example.html renamed to e2e/_unused/correct-example.html

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -57,8 +57,8 @@ <h1> Angular Best Practices </h1>
5757
</ul>
5858
</div>
5959

60-
<script src="../node_modules/angular/angular.js"></script>
61-
<script src="../dist/hint.js"></script>
60+
<script src="../../node_modules/angular/angular.js"></script>
61+
<script src="../../dist/hint.js"></script>
6262
<script>
6363
angular.module('bpAppModule', [])
6464
/**

examples/manual-bootstrap-alternative.html renamed to e2e/_unused/manual-bootstrap-alternative.html

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
<title>Angular Hint Example</title>
66

77
<script src="../../node_modules/angular/angular.js"></script>
8-
<script src="../../bower_components/angular-route/angular-route.js"></script>
8+
<script src="../../node_modules/angular-route/angular-route.js"></script>
99
<script src="../../dist/hint.js"></script>
1010
<script src="../util.js"></script>
1111

examples/manual-bootstrap.html renamed to e2e/_unused/manual-bootstrap.html

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
<title>Angular Hint Example</title>
66

77
<script src="../../node_modules/angular/angular.js"></script>
8-
<script src="../../bower_components/angular-route/angular-route.js"></script>
8+
<script src="../../node_modules/angular-route/angular-route.js"></script>
99
<script src="../../dist/hint.js"></script>
1010
<script src="../util.js"></script>
1111

examples/profile.html renamed to e2e/_unused/profile.html

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -34,8 +34,8 @@ <h1> Consuming ng-hint data </h1>
3434
</ul>
3535
</div>
3636

37-
<script src="../node_modules/angular/angular.js"></script>
38-
<script src="../dist/hint.js"></script>
37+
<script src="../../node_modules/angular/angular.js"></script>
38+
<script src="../../dist/hint.js"></script>
3939
<script>
4040
angular.module('profileApp', [])
4141
/**
File renamed without changes.

examples/exclude-wrong-module-name.spec.js renamed to e2e/exclude-wrong-module-name/exclude-wrong-module-name.spec.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
'use strict';
22

3-
var consoleText = require('./util.protractor');
3+
var consoleText = require('../util.protractor');
44

55
describe('angularHint', function() {
66
it('should warn if ng-hint is called with unknown options', function() {
File renamed without changes.

examples/hint.spec.js renamed to e2e/hint.spec.js

Lines changed: 6 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,10 @@
11
'use strict';
22

3+
var consoleText = require('./util.protractor');
4+
35
describe('angularHint', function() {
46
it('should include all modules by ng-hint default', function() {
5-
browser.get('allHint/');
7+
browser.get('all-hint/');
68

79
//angular-hint-controllers
810
expect(consoleText()).toContain('The best practice is to name ' +
@@ -29,14 +31,14 @@ describe('angularHint', function() {
2931

3032

3133
it('should have an inclusive mode', function() {
32-
browser.get('inclusiveHint/');
34+
browser.get('inclusive-hint/');
3335
//expect(consoleText())
3436
// .toContain('getElementById');
3537
});
3638

3739
// TODO: fix this assertion
3840
it('should have an exclusive mode', function() {
39-
browser.get('exclusiveHint/');
41+
browser.get('exclusive-hint/');
4042
//expect(consoleText()).toContain('getElementById');
4143
// expect(consoleText()).toContain('Variable "increment" called on DIV ' +
4244
// 'element does not exist in that scope.');
@@ -45,12 +47,8 @@ describe('angularHint', function() {
4547

4648

4749
it('should warn if there is no ng-hint attribute', function() {
48-
browser.get('noHint/');
50+
browser.get('no-hint/');
4951
expect(consoleText())
5052
.toBe('Angular Hint: General; Warning; ngHint is included on the page, but is not active because there is no `ng-hint` attribute present;');
5153
});
52-
53-
function consoleText() {
54-
return element(by.id('console')).getText();
55-
}
5654
});

examples/include-wrong-module-name.spec.js renamed to e2e/include-wrong-module-name/include-wrong-module-name.spec.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
'use strict';
22

3-
var consoleText = require('./util.protractor');
3+
var consoleText = require('../util.protractor');
44

55
describe('angularHint', function() {
66
it('should warn if ng-hint is called with unknown options', function() {
File renamed without changes.

e2e/no-hint/index.html

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,14 @@
1+
<!doctype html>
2+
<html ng-app>
3+
<head>
4+
<meta charset="utf-8">
5+
<title>Angular Hint Example</title>
6+
</head>
7+
<body>
8+
<div id="console"></div>
9+
10+
<script src="../../node_modules/angular/angular.js"></script>
11+
<script src="../../dist/hint.js"></script>
12+
<script src="../util.js"></script>
13+
</body>
14+
</html>
File renamed without changes.
File renamed without changes.

examples/strict-di.html

Whitespace-only changes.

0 commit comments

Comments
 (0)