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

test: fs pkglock lstatSync #40

Open
wants to merge 1 commit into
base: latest
Choose a base branch
from

Conversation

jchip
Copy link
Contributor

@jchip jchip commented Dec 1, 2018

  • first fs test that uses pkglock
  • need to add a process.tink._isSelf_ to by pass the self check

@jchip jchip force-pushed the pkglock-lstatsync-test branch from a0aad9f to a733499 Compare December 1, 2018 23:52
@jchip jchip force-pushed the pkglock-lstatsync-test branch from a733499 to f7d5260 Compare December 2, 2018 00:25
@jchip
Copy link
Contributor Author

jchip commented Dec 2, 2018

test was still passing with this commit:

commit bc20c7c68509be6b5d9f6c467fb61d7883f8c4b7
Author: Kat Marchán <[email protected]>
Date:   Wed Nov 28 20:26:27 2018 -0800

    refactor(node): move the extensions bit to its own module

2 new commits after that cause the test to fail with:

notice fs fetching [email protected]
ERR! prepare Error: No dependencies found matching filter
ERR! prepare     at Installer.prepare (/Users/xchen11/dev/pub/tink/lib/installer.js:6:1776)
ERR! prepare  { Error: No dependencies found matching filter
ERR! prepare     at Installer.prepare (/Users/xchen11/dev/pub/tink/lib/installer.js:6:1776)
ERR! prepare   stack:
ERR! prepare    'Error: No dependencies found matching filter\n    at Installer.prepare (/Users/xchen11/dev/pub/tink/lib/installer.js:6:1776)' }
test/node/fs-pkglock.js ............................... 0/2 1s
  lstatSync
  not ok fs-pkglock.lstatSync caught Error: No cache entry for tinked-package:2.0.0:require-from-path:1.0.2:https://registry.npmjs.org/require-from-path/-/require-from-path-1.0.2.tgz:sha512-hZrak6935a0RVpx6SFJg59IhRuwWkOBX4VPevL8iyxwoT8ep+ILMziLd1f6s5UCeZ6ckEk06xvfAe2I+3IpUaQ== found in /Users/xchen11/dev/pub/tink/test/.cache

@jchip
Copy link
Contributor Author

jchip commented Jan 8, 2019

@zkat any pointer in the error? I should get a few hours this/next week and will try to resolve this.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant