Skip to content
This repository has been archived by the owner on Jul 3, 2019. It is now read-only.

Commit

Permalink
test: issue with tests on windows
Browse files Browse the repository at this point in the history
  • Loading branch information
zkat committed Feb 5, 2017
1 parent f261c20 commit b586076
Showing 1 changed file with 4 additions and 5 deletions.
9 changes: 4 additions & 5 deletions test/registry.manifest.cache.js
Original file line number Diff line number Diff line change
Expand Up @@ -120,6 +120,10 @@ test('supports fetching from an optional cache', function (t) {
test('expires stale request data')
test('allows forcing use of cache when data stale')
test('falls back to registry if cache entry is invalid JSON')
// This test should prevent future footgunning if the caching logic changes
// accidentally. Caching manifests themselves should be entirely the job of the
// package fetcher.
test('does not insert plain manifests into the cache')

test('falls back to registry if cache entry missing', function (t) {
var opts = {
Expand All @@ -136,8 +140,3 @@ test('falls back to registry if cache entry missing', function (t) {
t.end()
})
})

// This test should prevent future footgunning if the caching logic changes
// accidentally. Caching manifests themselves should be entirely the job of the
// package fetcher.
test('does not insert plain manifests into the cache')

0 comments on commit b586076

Please sign in to comment.