chore(test): Add expectations.

By default, calling methods via `this` should not effect failures.
cartant · Jul 19, 2018 · 6d3aa9c · 6d3aa9c
1 parent 18abb3c
commit 6d3aa9c
Show file tree

Hide file tree

Showing 6 changed files with 29 additions and 0 deletions.
diff --git a/test/v5/fixtures/no-unsafe-scope/default/fixture.ts.lint b/test/v5/fixtures/no-unsafe-scope/default/fixture.ts.lint
@@ -67,7 +67,10 @@ class User {
                                                          ~~~~                       [no-unsafe-scope]
         of("Hello").pipe(map(value => `${value}, ${this.name}.`)).subscribe();
                                                    ~~~~                             [no-unsafe-scope]
+        Observable.of("Hello").map(value => `${value}, ${this.foo()}.`).subscribe();
+        of("Hello").pipe(map(value => `${value}, ${this.foo()}.`)).subscribe();
     }
+    foo(): string { return "foo"; }
 }
 
 [no-unsafe-scope]: Unsafe scopes are forbidden
diff --git a/test/v5/fixtures/no-unsafe-takewhile/default/fixture.ts.lint b/test/v5/fixtures/no-unsafe-takewhile/default/fixture.ts.lint
@@ -61,6 +61,13 @@ class Something {
             takeWhile(() => this._alive)
                             ~~~~                                        [no-unsafe-takewhile]
         ).subscribe();
+
+        _source.pipe(
+            takeWhile(() => this.alive())
+        ).subscribe();
+    }
+    alive(): boolean {
+        return this._alive;
     }
 }
 

diff --git a/test/v6-compat/fixtures/no-unsafe-scope/default/fixture.ts.lint b/test/v6-compat/fixtures/no-unsafe-scope/default/fixture.ts.lint
@@ -67,7 +67,10 @@ class User {
                                                          ~~~~                       [no-unsafe-scope]
         of("Hello").pipe(map(value => `${value}, ${this.name}.`)).subscribe();
                                                    ~~~~                             [no-unsafe-scope]
+        Observable.of("Hello").map(value => `${value}, ${this.foo()}.`).subscribe();
+        of("Hello").pipe(map(value => `${value}, ${this.foo()}.`)).subscribe();
     }
+    foo(): string { return "foo"; }
 }
 
 [no-unsafe-scope]: Unsafe scopes are forbidden
diff --git a/test/v6-compat/fixtures/no-unsafe-takewhile/default/fixture.ts.lint b/test/v6-compat/fixtures/no-unsafe-takewhile/default/fixture.ts.lint
@@ -61,6 +61,13 @@ class Something {
             takeWhile(() => this._alive)
                             ~~~~                                        [no-unsafe-takewhile]
         ).subscribe();
+
+        _source.pipe(
+            takeWhile(() => this.alive())
+        ).subscribe();
+    }
+    alive(): boolean {
+        return this._alive;
     }
 }
 

diff --git a/test/v6/fixtures/no-unsafe-scope/default/fixture.ts.lint b/test/v6/fixtures/no-unsafe-scope/default/fixture.ts.lint
@@ -44,7 +44,9 @@ class User {
     constructor(private name: string) {
         of("Hello").pipe(map(value => `${value}, ${this.name}.`)).subscribe();
                                                    ~~~~                             [no-unsafe-scope]
+        of("Hello").pipe(map(value => `${value}, ${this.foo()}.`)).subscribe();
     }
+    foo(): string { return "foo"; }
 }
 
 [no-unsafe-scope]: Unsafe scopes are forbidden
diff --git a/test/v6/fixtures/no-unsafe-takewhile/default/fixture.ts.lint b/test/v6/fixtures/no-unsafe-takewhile/default/fixture.ts.lint
@@ -40,6 +40,13 @@ class Something {
             takeWhile(() => this._alive)
                             ~~~~                                        [no-unsafe-takewhile]
         ).subscribe();
+
+        _source.pipe(
+            takeWhile(() => this.alive())
+        ).subscribe();
+    }
+    alive(): boolean {
+        return this._alive;
     }
 }