Skip to content

Commit

Permalink
test
Browse files Browse the repository at this point in the history
  • Loading branch information
julianmesa-gitkraken committed Jul 4, 2024
1 parent 13a84d9 commit 42cbb49
Show file tree
Hide file tree
Showing 5 changed files with 421 additions and 512 deletions.
78 changes: 40 additions & 38 deletions .github/workflows/tests.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -13,11 +13,13 @@ jobs:
name: "Linux Tests"
strategy:
matrix:
container: ["centos:7", "ubuntu:18.04"]
node: [12, 14]
container: ["ubuntu:18.04"]
node: [18, 20, 22]
runs-on: ubuntu-latest
container:
image: ${{ matrix.container }}
env:
ACTIONS_ALLOW_USE_UNSECURE_NODE_VERSION: true
steps:
- uses: actions/checkout@v2

Expand All @@ -43,44 +45,44 @@ jobs:
yarn
yarn test
macos-tests:
name: "MacOS Tests"
strategy:
matrix:
node: [12, 14]
runs-on: macOS-latest
steps:
- uses: actions/checkout@v2
# macos-tests:
# name: "MacOS Tests"
# strategy:
# matrix:
# node: [16, 18, 20, 22]
# runs-on: macOS-latest
# steps:
# - uses: actions/checkout@v4

- name: Use Node.js ${{ matrix.node }}
uses: actions/setup-node@v2
with:
node-version: ${{ matrix.node }}
check-latest: true
# - name: Use Node.js ${{ matrix.node }}
# uses: actions/setup-node@v4
# with:
# node-version: ${{ matrix.node }}
# check-latest: true

- run: |
npm install --global yarn
yarn
yarn test
# - run: |
# npm install --global yarn
# yarn
# yarn test

windows-tests:
name: "Windows Tests"
strategy:
matrix:
node: [12, 14]
runs-on: windows-latest
steps:
- uses: actions/checkout@v2
# windows-tests:
# name: "Windows Tests"
# strategy:
# matrix:
# node: [12, 14]
# runs-on: windows-latest
# steps:
# - uses: actions/checkout@v4

- name: Use Node.js ${{ matrix.node }}
uses: actions/setup-node@v2
with:
node-version: ${{ matrix.node }}
check-latest: true
# - name: Use Node.js ${{ matrix.node }}
# uses: actions/setup-node@v4
# with:
# node-version: ${{ matrix.node }}
# check-latest: true

- run: |
npm install --global yarn
npm install --global [email protected]
npm prefix -g | % {npm config set node_gyp "$_\node_modules\node-gyp\bin\node-gyp.js"}
yarn
yarn test
# - run: |
# npm install --global yarn
# npm install --global [email protected]
# npm prefix -g | % {npm config set node_gyp "$_\node_modules\node-gyp\bin\node-gyp.js"}
# yarn
# yarn test
4 changes: 2 additions & 2 deletions binding.gyp
Original file line number Diff line number Diff line change
Expand Up @@ -29,10 +29,10 @@
],
"msvs_settings": {
"VCCLCompilerTool": {
"DisableSpecificWarnings": [ "4506", "4538", "4793" ]
"DisableSpecificWarnings": [ ]
},
"VCLinkerTool": {
"AdditionalOptions": [ "/ignore:4248" ]
"AdditionalOptions": [ ]
}
}
}],
Expand Down
6 changes: 3 additions & 3 deletions includes/win32/Watcher.h
Original file line number Diff line number Diff line change
Expand Up @@ -37,10 +37,10 @@ class Watcher
void resizeBuffers(std::size_t size);

std::string getUTF8Directory(std::wstring path) ;
bool Watcher::isExcluded(const std::wstring &fileName);
bool isExcluded(const std::wstring &fileName);

std::wstring Watcher::getWatchedPathFromHandle();
void Watcher::checkWatchedPath();
std::wstring getWatchedPathFromHandle();
void checkWatchedPath();

std::atomic<bool> mRunning;
SingleshotSemaphore mHasStartedSemaphore;
Expand Down
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@
],
"homepage": "https:/axosoft/node-simple-file-watcher",
"dependencies": {
"node-addon-api": "^5.0.0"
"node-addon-api": "*"
},
"devDependencies": {
"eslint": "^6.8.0",
Expand Down
Loading

0 comments on commit 42cbb49

Please sign in to comment.