Github user stevengill commented on a diff in the pull request:
https://github.com/apache/cordova-lib/pull/510#discussion_r95500848
--- Diff: cordova-lib/spec-cordova/plugin.spec.js ---
@@ -216,91 +229,110 @@ describe('plugin end-to-end', function() {
expect(defaultPluginPreferences.REQUIRED).toBe('NO');
return removePlugin(org_test_defaultvariables);
})
- .fail(errorHandler.errorCallback)
- .fin(done);
- });
+ .fail(function(err) {
+ console.error(err);
+ expect(err).toBeUndefined();
+ })
+ .fin(done);
+ }, 30000);
- it('should successfully add a plugin when specifying CLI variables',
function(done) {
+ it('Test 006 : should successfully add a plugin when specifying CLI
variables', function(done) {
addPlugin(path.join(pluginsDir, org_test_defaultvariables),
org_test_defaultvariables, {cli_variables: { REQUIRED:'yes',
REQUIRED_ANDROID:'yes'}}, done)
- .fail(errorHandler.errorCallback)
+ .fail(function(err) {
+ console.error(err);
+ expect(err).toBeUndefined();
+ })
.fin(done);
- });
+ }, 30000);
- it('should not check npm info when using the searchpath flag',
function(done) {
+ it('Test 007 : should not check npm info when using the searchpath
flag', function(done) {
mockPluginFetch(npmInfoTestPlugin, path.join(pluginsDir,
npmInfoTestPlugin));
-
spyOn(registry, 'info');
- addPlugin(npmInfoTestPlugin, npmInfoTestPlugin, {searchpath:
pluginsDir}, done)
+ return addPlugin(npmInfoTestPlugin, npmInfoTestPlugin,
{searchpath: pluginsDir}, done)
.then(function() {
expect(registry.info).not.toHaveBeenCalled();
-
- var fetchOptions = plugman.raw.fetch.mostRecentCall.args[2];
- expect(fetchOptions.searchpath).toBeDefined();
+ var fetchOptions =
plugman.raw.fetch.calls.mostRecent().args[2];
+ expect(fetchOptions.searchpath[0]).toExist();
+ })
+ .fail(function(err) {
+ console.error(err);
+ expect(err).toBeUndefined();
})
- .fail(errorHandler.errorCallback)
.fin(done);
- });
+ }, 30000);
- it('should not check npm info when using the noregistry flag',
function(done) {
+ it('Test 008 : should not check npm info when using the noregistry
flag', function(done) {
mockPluginFetch(npmInfoTestPlugin, path.join(pluginsDir,
npmInfoTestPlugin));
spyOn(registry, 'info');
addPlugin(npmInfoTestPlugin, npmInfoTestPlugin, {noregistry:true},
done)
.then(function() {
expect(registry.info).not.toHaveBeenCalled();
- var fetchOptions = plugman.raw.fetch.mostRecentCall.args[2];
+ var fetchOptions =
plugman.raw.fetch.calls.mostRecent().args[2];
expect(fetchOptions.noregistry).toBeTruthy();
})
- .fail(errorHandler.errorCallback)
+ .fail(function(err) {
+ console.error(err);
+ expect(err).toBeUndefined();
+ })
.fin(done);
- });
+ }, 30000);
- it('should not check npm info when fetching from a Git repository',
function(done) {
+ it('Test 009 : should not check npm info when fetching from a Git
repository', function(done) {
spyOn(registry, 'info');
addPlugin(testGitPluginRepository, testGitPluginId, {}, done)
.then(function() {
expect(registry.info).not.toHaveBeenCalled();
})
- .fail(errorHandler.errorCallback)
+ .fail(function(err) {
+ console.error(err);
--- End diff --
same as 155
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
