Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Investigate failing NSP tests #2511

Closed
paulmelnikow opened this issue Dec 11, 2018 · 4 comments · Fixed by #2529
Closed

Investigate failing NSP tests #2511

paulmelnikow opened this issue Dec 11, 2018 · 4 comments · Fixed by #2529
Labels
bug Bugs in badges and the frontend keep-service-tests-green Related to fixing failing tests of the services service-badge Accepted and actionable changes, features, and bugs

Comments

@paulmelnikow
Copy link
Member

NSP shut down; maybe the API is gone?

  5) Node Security Platform
       Format '/nsp/npm/:package/:version.:format' where it doesn't exist
         
	[ GET /npm/some-no-exist/1.0.0.json ]:

      AssertionError: expected { name: 'nsp', value: 'invalid' } to deeply equal { Object (name, value) }
      + expected - actual

       {
         "name": "nsp"
      -  "value": "invalid"
      +  "value": "no known vulnerabilities"
       }
      
      at Object.pathMatch.matchJSON (node_modules/icedfrisby/lib/pathMatch.js:138:38)
      at _expect (node_modules/icedfrisby/lib/icedfrisby.js:583:10)
      at IcedFrisbyNock._invokeExpects (node_modules/icedfrisby/lib/icedfrisby.js:1261:26)
      at start (node_modules/icedfrisby/lib/icedfrisby.js:1244:12)
      at Request.runCallback [as _callback] (node_modules/icedfrisby/lib/icedfrisby.js:1131:16)
      at Request.self.callback (node_modules/request/request.js:185:22)
      at Request.<anonymous> (node_modules/request/request.js:1161:10)
      at IncomingMessage.<anonymous> (node_modules/request/request.js:1083:12)
      at endReadableNT (_stream_readable.js:1064:12)
      at _combinedTickCallback (internal/process/next_tick.js:139:11)
      at process._tickDomainCallback (internal/process/next_tick.js:219:9)

  6) Node Security Platform
       Format '/nsp/npm/:package/:version.:format' without vulnerabilities
         
	[ GET /npm/bronze/1.4.0.json ]:

      AssertionError: expected { name: 'nsp', value: 'invalid' } to deeply equal { Object (name, value) }
      + expected - actual

       {
         "name": "nsp"
      -  "value": "invalid"
      +  "value": "no known vulnerabilities"
       }
      
      at Object.pathMatch.matchJSON (node_modules/icedfrisby/lib/pathMatch.js:138:38)
      at _expect (node_modules/icedfrisby/lib/icedfrisby.js:583:10)
      at IcedFrisbyNock._invokeExpects (node_modules/icedfrisby/lib/icedfrisby.js:1261:26)
      at start (node_modules/icedfrisby/lib/icedfrisby.js:1244:12)
      at Request.runCallback [as _callback] (node_modules/icedfrisby/lib/icedfrisby.js:1131:16)
      at Request.self.callback (node_modules/request/request.js:185:22)
      at Request.<anonymous> (node_modules/request/request.js:1161:10)
      at IncomingMessage.<anonymous> (node_modules/request/request.js:1083:12)
      at endReadableNT (_stream_readable.js:1064:12)
      at _combinedTickCallback (internal/process/next_tick.js:139:11)
      at process._tickDomainCallback (internal/process/next_tick.js:219:9)

  7) Node Security Platform
       Format '/nsp/npm/@:scope/:package/:version.:format' where it doesn't exist
         
	[ GET /npm/@some-no-exist/some-no-exist/1.0.0.json ]:

      AssertionError: expected { name: 'nsp', value: 'invalid' } to deeply equal { Object (name, value) }
      + expected - actual

       {
         "name": "nsp"
      -  "value": "invalid"
      +  "value": "no known vulnerabilities"
       }
      
      at Object.pathMatch.matchJSON (node_modules/icedfrisby/lib/pathMatch.js:138:38)
      at _expect (node_modules/icedfrisby/lib/icedfrisby.js:583:10)
      at IcedFrisbyNock._invokeExpects (node_modules/icedfrisby/lib/icedfrisby.js:1261:26)
      at start (node_modules/icedfrisby/lib/icedfrisby.js:1244:12)
      at Request.runCallback [as _callback] (node_modules/icedfrisby/lib/icedfrisby.js:1131:16)
      at Request.self.callback (node_modules/request/request.js:185:22)
      at Request.<anonymous> (node_modules/request/request.js:1161:10)
      at IncomingMessage.<anonymous> (node_modules/request/request.js:1083:12)
      at endReadableNT (_stream_readable.js:1064:12)
      at _combinedTickCallback (internal/process/next_tick.js:139:11)
      at process._tickDomainCallback (internal/process/next_tick.js:219:9)

  8) Node Security Platform
       Format '/nsp/npm/@:scope/:package/:version.:format' without vulnerabilities
         
	[ GET /npm/@cycle/core/1.0.0.json ]:

      AssertionError: expected { name: 'nsp', value: 'invalid' } to deeply equal { Object (name, value) }
      + expected - actual

       {
         "name": "nsp"
      -  "value": "invalid"
      +  "value": "no known vulnerabilities"
       }
      
      at Object.pathMatch.matchJSON (node_modules/icedfrisby/lib/pathMatch.js:138:38)
      at _expect (node_modules/icedfrisby/lib/icedfrisby.js:583:10)
      at IcedFrisbyNock._invokeExpects (node_modules/icedfrisby/lib/icedfrisby.js:1261:26)
      at start (node_modules/icedfrisby/lib/icedfrisby.js:1244:12)
      at Request.runCallback [as _callback] (node_modules/icedfrisby/lib/icedfrisby.js:1131:16)
      at Request.self.callback (node_modules/request/request.js:185:22)
      at Request.<anonymous> (node_modules/request/request.js:1161:10)
      at IncomingMessage.<anonymous> (node_modules/request/request.js:1083:12)
      at endReadableNT (_stream_readable.js:1064:12)
      at _combinedTickCallback (internal/process/next_tick.js:139:11)
      at process._tickDomainCallback (internal/process/next_tick.js:219:9)
@paulmelnikow paulmelnikow added bug Bugs in badges and the frontend service-badge Accepted and actionable changes, features, and bugs keep-service-tests-green Related to fixing failing tests of the services labels Dec 11, 2018
@calebcartwright
Copy link
Member

That's funny timing. I was just noticing the nsp service directory in my IDE and wondering if it still worked. The main website @ https://nodesecurity.io appears to have finally been shuttered (redirecting to the npm registry now), and the API endpoint correspondingly seems dead as well https://api.nodesecurity.io/

@calebcartwright
Copy link
Member

My assumption is that this badge will need to be deprecated given that the nsp service no longer exists. Is there a standard process for badge deprecation?

Obviously it could be completely removed, but I can see value in a phased removal process as well (first change the badge example docs to indicate deprecation, update the badge to always return something like nsp | deprecated etc.)

@paulmelnikow
Copy link
Member Author

Yup! Here's a recent example of how we handle these: #2352.

@calebcartwright
Copy link
Member

Perfect thanks. I can take this one

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Bugs in badges and the frontend keep-service-tests-green Related to fixing failing tests of the services service-badge Accepted and actionable changes, features, and bugs
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants