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

Add ECSql tests for ECDb (#7323) #7583

Merged
merged 11 commits into from
Jan 23, 2025
Original file line number Diff line number Diff line change
@@ -0,0 +1,10 @@
{
"changes": [
{
"packageName": "@itwin/core-backend",
"comment": "",
"type": "none"
}
],
"packageName": "@itwin/core-backend"
}
9 changes: 5 additions & 4 deletions common/config/azure-pipelines/templates/core-build.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -102,10 +102,11 @@ steps:
IMJS_OIDC_CLIENT_SECRET: $(IMJS_OIDC_CLIENT_SECRET)
condition: and(succeeded(), ${{ parameters.buildMobile }}, eq(variables['Agent.OS'], 'Darwin'))

- script: npm run ios:all
workingDirectory: core/backend
displayName: Build & run iOS backend unit tests in Simulator
condition: and(succeeded(), ${{ parameters.buildMobile }}, eq(variables['Agent.OS'], 'Darwin'))
# Tests seem broken on ci, contacted Travis.Cobby to investigate
aruniverse marked this conversation as resolved.
Show resolved Hide resolved
# - script: npm run ios:all
# workingDirectory: core/backend
# displayName: Build & run iOS backend unit tests in Simulator
# condition: and(succeeded(), ${{ parameters.buildMobile }}, eq(variables['Agent.OS'], 'Darwin'))

- script: node common/scripts/install-run-rush.js lint
displayName: rush lint
Expand Down
8 changes: 8 additions & 0 deletions common/config/rush/browser-approved-packages.json
Original file line number Diff line number Diff line change
Expand Up @@ -670,6 +670,10 @@
"name": "lodash",
"allowedCategories": [ "backend", "tools" ]
},
{
"name": "marked",
"allowedCategories": [ "backend" ]
},
{
"name": "memfs",
"allowedCategories": [ "tools" ]
Expand Down Expand Up @@ -826,6 +830,10 @@
"name": "spdy",
"allowedCategories": [ "internal" ]
},
{
"name": "sql-formatter",
"allowedCategories": [ "backend" ]
},
{
"name": "stream-browserify",
"allowedCategories": [ "integration-testing", "internal" ]
Expand Down
59 changes: 59 additions & 0 deletions common/config/rush/pnpm-lock.yaml

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

5 changes: 4 additions & 1 deletion core/backend/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -9,11 +9,12 @@
"node": "^18.0.0 || ^20.0.0 || ^22.0.0"
},
"scripts": {
"build": "npm run -s build:cjs && npm run -s copy:assets && npm run -s copy:test-assets",
"build": "npm run -s build:cjs && npm run -s copy:assets && npm run -s copy:test-assets && npm run -s copy:ecsqltestmdfiles",
"build:cjs": "tsc 1>&2 --outDir lib/cjs",
"clean": "rimraf lib .rush/temp/package-deps*.json ../../tools/internal/ios/core-test-runner/build ../../tools/internal/lib",
"docs": "betools docs --json=../../generated-docs/core/core-backend/file.json --tsIndexFile=./core-backend.ts --onlyJson --excludeGlob=**/*.d.ts",
"copy:assets": "cpx \"./src/assets/**/*\" ./lib/cjs/assets",
"copy:ecsqltestmdfiles": "cpx \"./src/test/ecsql/queries/*\" ./lib/cjs/test/ecsql/queries",
"copy:config": "internal-tools copy-config",
"copy:test-assets": "cpx \"./src/test/assets/**/*\" ./lib/cjs/test/assets",
"cover": "nyc npm -s test",
Expand Down Expand Up @@ -86,13 +87,15 @@
"eslint": "^9.13.0",
"fs-extra": "^8.1.0",
"internal-tools": "workspace:*",
"marked": "^14.1.3",
"mocha": "^10.2.0",
"node-simctl": "~7.6.1",
"null-loader": "^4.0.1",
"nyc": "^15.1.0",
"rimraf": "^3.0.2",
"sinon": "^17.0.2",
"source-map-loader": "^4.0.0",
"sql-formatter": "^15.4.6",
"typescript": "~5.6.2",
"webpack": "^5.76.0"
},
Expand Down
Loading
Loading