Skip to content

Commit

Permalink
Add ECSql tests for ECDb (#7323) (#7583)
Browse files Browse the repository at this point in the history
Co-authored-by: Robert Schili <rschili@users.noreply.github.com>
  • Loading branch information
soham-bentley and rschili authored Jan 23, 2025
1 parent c0b1aff commit 8d197c5
Show file tree
Hide file tree
Showing 36 changed files with 12,660 additions and 5 deletions.
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
# - 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

0 comments on commit 8d197c5

Please sign in to comment.