feat(upgrade): return a function (instead of array) from `downgradeInjectable()` (#14037)
This makes it more consistent with the dynamic version of `upgrade` and makes it
possible to share code between the dynamic and static versions.
This commit also refactors the file layout, moving common and dynamic-specific
files to `common/` and `dynamic/` directories respectively and renaming `aot/`
to `static/`.
Some private keys, used as AngularJS DI tokens, have also been renamed, but this
should not affect apps, since these keys are undocumented and not supposed to
be used externally.
BREAKING CHANGE:
Previously, `upgrade/static/downgradeInjectable` returned an array of the form:
```js
['dep1', 'dep2', ..., function factory(dep1, dep2, ...) { ... }]
```
Now it returns a function with an `$inject` property:
```js
factory.$inject = ['dep1', 'dep2', ...];
function factory(dep1, dep2, ...) { ... }
```
It shouldn't affect the behavior of apps, since both forms are equally suitable
to be used for registering AngularJS injectable services, but it is possible
that type-checking might fail or that current code breaks if it relies on the
returned value being an array.
2017-01-13 09:20:28 -05:00
|
|
|
/**
|
|
|
|
* @license
|
|
|
|
* Copyright Google Inc. All Rights Reserved.
|
|
|
|
*
|
|
|
|
* Use of this source code is governed by an MIT-style license that can be
|
|
|
|
* found in the LICENSE file at https://angular.io/license
|
|
|
|
*/
|
2018-02-15 12:21:18 -05:00
|
|
|
import {setAngularJSGlobal} from '@angular/upgrade/src/common/angular1';
|
|
|
|
|
|
|
|
|
|
|
|
const ng1Versions = [
|
|
|
|
{
|
|
|
|
label: '1.5',
|
|
|
|
files: ['angular-1.5/angular.js', 'angular-mocks-1.5/angular-mocks.js'],
|
|
|
|
},
|
|
|
|
{
|
|
|
|
label: '1.6',
|
|
|
|
files: ['angular/angular.js', 'angular-mocks/angular-mocks.js'],
|
|
|
|
},
|
|
|
|
];
|
|
|
|
|
|
|
|
export function createWithEachNg1VersionFn(setNg1: typeof setAngularJSGlobal) {
|
|
|
|
return (specSuite: () => void) => ng1Versions.forEach(({label, files}) => {
|
|
|
|
describe(`[AngularJS v${label}]`, () => {
|
|
|
|
beforeAll(done => {
|
|
|
|
// Load AngularJS before running tests.
|
|
|
|
files
|
|
|
|
.reduce(
|
|
|
|
(prev, file) => prev.then(() => new Promise<void>((resolve, reject) => {
|
|
|
|
const script = document.createElement('script');
|
|
|
|
script.src = `base/node_modules/${file}`;
|
|
|
|
script.onerror = reject;
|
|
|
|
script.onload = () => {
|
|
|
|
document.body.removeChild(script);
|
|
|
|
resolve();
|
|
|
|
};
|
|
|
|
document.body.appendChild(script);
|
|
|
|
})),
|
|
|
|
Promise.resolve())
|
|
|
|
.then(() => setNg1((window as any).angular))
|
|
|
|
.then(done, done.fail);
|
|
|
|
});
|
|
|
|
|
|
|
|
afterAll(() => {
|
|
|
|
// In these tests we are loading different versions of AngularJS on the same window.
|
|
|
|
// AngularJS leaves an "expandoId" property on `document`, which can trick subsequent
|
|
|
|
// `window.angular` instances into believing an app is already bootstrapped.
|
|
|
|
(window as any).angular.element(document).removeData();
|
|
|
|
|
|
|
|
// Remove AngularJS to leave a clean state for subsequent tests.
|
|
|
|
setNg1(undefined);
|
|
|
|
delete (window as any).angular;
|
|
|
|
});
|
|
|
|
|
|
|
|
specSuite();
|
|
|
|
});
|
|
|
|
});
|
|
|
|
}
|
feat(upgrade): return a function (instead of array) from `downgradeInjectable()` (#14037)
This makes it more consistent with the dynamic version of `upgrade` and makes it
possible to share code between the dynamic and static versions.
This commit also refactors the file layout, moving common and dynamic-specific
files to `common/` and `dynamic/` directories respectively and renaming `aot/`
to `static/`.
Some private keys, used as AngularJS DI tokens, have also been renamed, but this
should not affect apps, since these keys are undocumented and not supposed to
be used externally.
BREAKING CHANGE:
Previously, `upgrade/static/downgradeInjectable` returned an array of the form:
```js
['dep1', 'dep2', ..., function factory(dep1, dep2, ...) { ... }]
```
Now it returns a function with an `$inject` property:
```js
factory.$inject = ['dep1', 'dep2', ...];
function factory(dep1, dep2, ...) { ... }
```
It shouldn't affect the behavior of apps, since both forms are equally suitable
to be used for registering AngularJS injectable services, but it is possible
that type-checking might fail or that current code breaks if it relies on the
returned value being an array.
2017-01-13 09:20:28 -05:00
|
|
|
|
|
|
|
export function html(html: string): Element {
|
|
|
|
// Don't return `body` itself, because using it as a `$rootElement` for ng1
|
|
|
|
// will attach `$injector` to it and that will affect subsequent tests.
|
|
|
|
const body = document.body;
|
|
|
|
body.innerHTML = `<div>${html.trim()}</div>`;
|
|
|
|
const div = document.body.firstChild as Element;
|
|
|
|
|
|
|
|
if (div.childNodes.length === 1 && div.firstChild instanceof HTMLElement) {
|
|
|
|
return div.firstChild;
|
|
|
|
}
|
|
|
|
|
|
|
|
return div;
|
|
|
|
}
|
|
|
|
|
2017-07-05 06:58:27 -04:00
|
|
|
export function multiTrim(text: string | null | undefined, allSpace = false): string {
|
2017-03-29 12:34:45 -04:00
|
|
|
if (typeof text == 'string') {
|
2017-07-05 06:58:27 -04:00
|
|
|
const repl = allSpace ? '' : ' ';
|
|
|
|
return text.replace(/\n/g, '').replace(/\s+/g, repl).trim();
|
2017-03-29 12:34:45 -04:00
|
|
|
}
|
|
|
|
throw new Error('Argument can not be undefined.');
|
feat(upgrade): return a function (instead of array) from `downgradeInjectable()` (#14037)
This makes it more consistent with the dynamic version of `upgrade` and makes it
possible to share code between the dynamic and static versions.
This commit also refactors the file layout, moving common and dynamic-specific
files to `common/` and `dynamic/` directories respectively and renaming `aot/`
to `static/`.
Some private keys, used as AngularJS DI tokens, have also been renamed, but this
should not affect apps, since these keys are undocumented and not supposed to
be used externally.
BREAKING CHANGE:
Previously, `upgrade/static/downgradeInjectable` returned an array of the form:
```js
['dep1', 'dep2', ..., function factory(dep1, dep2, ...) { ... }]
```
Now it returns a function with an `$inject` property:
```js
factory.$inject = ['dep1', 'dep2', ...];
function factory(dep1, dep2, ...) { ... }
```
It shouldn't affect the behavior of apps, since both forms are equally suitable
to be used for registering AngularJS injectable services, but it is possible
that type-checking might fail or that current code breaks if it relies on the
returned value being an array.
2017-01-13 09:20:28 -05:00
|
|
|
}
|
2017-03-13 20:34:53 -04:00
|
|
|
|
|
|
|
export function nodes(html: string) {
|
|
|
|
const div = document.createElement('div');
|
|
|
|
div.innerHTML = html.trim();
|
|
|
|
return Array.prototype.slice.call(div.childNodes);
|
|
|
|
}
|
2018-02-15 12:21:18 -05:00
|
|
|
|
|
|
|
export const withEachNg1Version = createWithEachNg1VersionFn(setAngularJSGlobal);
|