Skip to content
This repository was archived by the owner on Dec 8, 2022. It is now read-only.

Commit 16644d2

Browse files
author
Bobby Earl
authored
Fixed deps + test (#171)
* Fixed deps + test * Added new test + note * Updated test comment and package.json. Added entry for CHANGELOG.md
1 parent 8ba98a0 commit 16644d2

5 files changed

+19
-3
lines changed

CHANGELOG.md

+4
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,9 @@
11
# 1.0.0-beta.29 (2017-06-05)
22

3+
- Bugfix for `SkyAuthHttp`. [#171](https://github.com/blackbaud/skyux-builder/pull/171)
4+
5+
# 1.0.0-beta.29 (2017-06-05)
6+
37
- Implemented style loader to resolve FOUC (flash of unstyled content). [#166](https://github.com/blackbaud/skyux-builder/pull/166)
48
- Initial creation of `skyRouterLink` directive. [#159](https://github.com/blackbaud/skyux-builder/pull/159)
59
- Updated testing suite to include internal files in `src/app` directory.

lib/sky-pages-module-generator.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,7 @@ function getSource(skyAppConfig) {
6060
runtimeProviders.push(`{
6161
provide: SkyAuthHttp,
6262
useClass: SkyAuthHttp,
63-
deps: [XHRBackend, RequestOptions, SkyAppWindowRef, SkyAuthTokenProvider]
63+
deps: [XHRBackend, RequestOptions, SkyAuthTokenProvider, SkyAppConfig]
6464
}`);
6565
}
6666

package.json

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "@blackbaud/skyux-builder",
3-
"version": "1.0.0-beta.29",
3+
"version": "1.0.0-beta.30",
44
"description": "Builds the output of a SKY UX application.",
55
"main": "index.js",
66
"scripts": {

runtime/auth-http.spec.ts

+12
Original file line numberDiff line numberDiff line change
@@ -128,4 +128,16 @@ describe('SkyAuthHttp', () => {
128128
});
129129
});
130130

131+
/**
132+
* PLEASE NOTE
133+
* If this tests fails, it means you've changed the required parameters to the constructor.
134+
* To successfully maintain backwards compatibility:
135+
* - make any new parameters optional
136+
* - add any new parameters to the end of the constructor
137+
*/
138+
it('should maintain backwards compatibility if new parameters added to the constructor', () => {
139+
setupInjector('');
140+
expect(skyAuthHttp).toBeDefined();
141+
});
142+
131143
});

test/sky-pages-module-generator.spec.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,7 @@ describe('SKY UX Builder module generator', () => {
7878
const expectedProvider = `{
7979
provide: SkyAuthHttp,
8080
useClass: SkyAuthHttp,
81-
deps: [XHRBackend, RequestOptions, SkyAppWindowRef, SkyAuthTokenProvider]
81+
deps: [XHRBackend, RequestOptions, SkyAuthTokenProvider, SkyAppConfig]
8282
}`;
8383

8484
let source = generator.getSource({

0 commit comments

Comments
 (0)