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

@bentley/imodeljs-native 3.3.3 #3832

Merged
merged 33 commits into from
Jun 21, 2022
Merged

@bentley/imodeljs-native 3.3.3 #3832

merged 33 commits into from
Jun 21, 2022

Conversation

pmconne
Copy link
Member

@pmconne pmconne commented Jun 18, 2022

Locked down ts-node to 10.8.0 because this bug surfaced in core-common cover script failing windows builds with path not found and causing upload of coverage report to fail on macOS.

Locked down superagent to 7.1.3 because 7.1.6 produced following backend-integration-test failures in electron only:

  1) Background map (#integration)
       obscures model based on settings:

      AssertionError: expected true to equal false
      + expected - actual

      -true
      +false
      
      at expectColor (home/vsts/work/1/s/full-stack-tests/core/src/frontend/map/BackgroundMap.test.ts:69:72)
      at expectPixelTypes (home/vsts/work/1/s/full-stack-tests/core/src/frontend/map/BackgroundMap.test.ts:73:7)
      at home/vsts/work/1/s/full-stack-tests/core/src/frontend/map/BackgroundMap.test.ts:117:9
      at testOnScreenViewport (home/vsts/work/1/s/full-stack-tests/core/src/frontend/TestViewport.ts:336:5)
      at Context.<anonymous> (home/vsts/work/1/s/full-stack-tests/core/src/frontend/map/BackgroundMap.test.ts:112:5)

  2) RealityDataAccess (#integration)
       should be able to call getPublisherProductInfo on RealityDataSource for all supported displayable reality data in itwin project:
     Error: Browser-only version of superagent could not find XHR
      at parse (home/vsts/work/1/s/core/frontend/src/request/Request.ts:169:19)
      at request (home/vsts/work/1/s/core/frontend/src/request/Request.ts:424:25)
      at RealityDataSourceContextShareImpl.getRealityDataTileJson (home/vsts/work/1/s/core/frontend/src/RealityDataSourceContextShareImpl.ts:119:18)
      at RealityDataSourceContextShareImpl.getPublisherProductInfo (home/vsts/work/1/s/core/frontend/src/RealityDataSourceContextShareImpl.ts:248:28)
      at Context.<anonymous> (home/vsts/work/1/s/full-stack-tests/core/src/frontend/standalone/RealityDataAccess.test.ts:192:23)

  3) RealityDataAccess (#integration)
       should be able to call getFileInfo when RealityDataSource is a 3dTile reality data:
     Error: Browser-only version of superagent could not find XHR
      at parse (home/vsts/work/1/s/core/frontend/src/request/Request.ts:169:19)
      at request (home/vsts/work/1/s/core/frontend/src/request/Request.ts:424:25)
      at RealityDataSourceContextShareImpl.getRealityDataTileJson (home/vsts/work/1/s/core/frontend/src/RealityDataSourceContextShareImpl.ts:119:18)
      at Context.<anonymous> (home/vsts/work/1/s/full-stack-tests/core/src/frontend/standalone/RealityDataAccess.test.ts:211:32)

  4) RealityDataAccess (#integration)
       should be able to call getSpatialLocationAndExtents on RealityDataSource for all supported displayable reality data in itwin project:
     Error: Browser-only version of superagent could not find XHR
      at parse (home/vsts/work/1/s/core/frontend/src/request/Request.ts:169:19)
      at request (home/vsts/work/1/s/core/frontend/src/request/Request.ts:424:25)
      at RealityDataSourceContextShareImpl.getRealityDataTileJson (home/vsts/work/1/s/core/frontend/src/RealityDataSourceContextShareImpl.ts:119:18)
      at RealityDataSourceContextShareImpl.getSpatialLocationAndExtents (home/vsts/work/1/s/core/frontend/src/RealityDataSourceContextShareImpl.ts:221:28)
      at Context.<anonymous> (home/vsts/work/1/s/full-stack-tests/core/src/frontend/standalone/RealityDataAccess.test.ts:228:33)

pmconne and others added 14 commits May 12, 2022 17:03
* improve exhaustive transformer test (#3578)

* correct gathering of properties to include base properties in assertIdentityTransformation
* add an option to compare element geometry
* fix more jsonProperties remaps

Co-authored-by: Michael Belousov <MichaelBelousov@users.noreply.github.com>

* Add option defaultTool prop to StandardFrontstageProps (#3583)

* Allow apps to override the default tool in a
frontstage created by the StandardFrontstageProvider

* rush change

* 3.2.0-dev.64

* UI: Fix visibility tree checkbox alignment (#3575)

* 3.2.0-dev.65

* 3.2.0-dev.66

* Viewport.readPixels returns undefined if viewport is disposed. (#3592)

* 3.2.0-dev.67

* 3.2.0-dev.68

* 3.2.0-dev.69

* Fix hypermodeling popup toolbar (#3595)

* Clear timeout on mouse leave; don't queue timeout when picking an overlay.

* Double delay before toolbar closes.

* Update some test and tooling dependencies (#3108)

* update dev dependencies
* add missing types dependency
* fix ui tests
* node 15+ hangs when using MessageChannel & jsdom - fix

Co-authored-by: Arun George <aruniverse@users.noreply.github.com>
Co-authored-by: johnnyd710 <19596966+johnnyd710@users.noreply.github.com>

* Element Tooltip Blocks Widgets (#3596)

* Add event listender to close ElementToolTip if the
mouse leaves the canvas.

* Add comments to explain mousemove listener.

* rush change

* Fix for losing viewport content after clicking PW link (#3600)

* Use unload not beforeUnload event for UI clean up.

* update test to send unload event

* rush change

* Support multiple features per decoration graphic (#3598)

* wip

* GraphicBuilder.activateFeature

* VertexKey has an optional Feature.

* Initialize feature in constructor.

* Feature (sans tests).

* adjust core-frontend tests (they all pass).

* Add multi-feature decoration to display-test-app example.

* test.

* docs.

* wip nextversion

* Make test actually batch features...

* NextVersion

* update test

* timeout no work on describe?

* extract-api

* lint

* deprecation message.

* Adding a new abstract method is a breaking API change.

* relative doc links.

* Fix corrupted background map textures (#3599)

* Map tiles need to retain their textures longer than reality mesh geometry will if it disposes of everything when requested. These changes make reality meshes that are maps hold onto their textures.

* rush change

* clean up RealityMeshGeometry constructor by using a props object. Make reality mesh System methods use default value of false for _disableTextureDisposal.

* No changelog needed...

...for a regression introduced and fixed with no intervening release.

Co-authored-by: Paul Connelly <22944042+pmconne@users.noreply.github.com>

* 3.2.0-dev.70

* ESRI OAuth2 support in map-layers (#1677)

* WIP ESRI OAuth2 support.

* WIP

* Improved UI

* WIP

* WIP

* Correctly persist the token in browser storage.  Code refactoring + testing.

* Renamed EsriOAuth2.getOAuth2EndpointFromRestUrl to EsriOAuth2.getOAuth2EndpointFromMapLayerUrl

* No longer attempt to used expired tokens.  Code refactoring / cleanup.

* Fix previous tests and added improved it.  Few other fixes.

* Improved Oauth2 test.

* Renamed useEsriOauth2Popup to usePopup.

* Moved usePopup and useInterval inside ui-framework

* Moved usePopup and useInterval inside ui-framework (missing file)

* Added beta release tag to EsriOauth2 + refactoring + lint

* Moved parsing logic from the callback code to the EsriOauth2 class.

* fixed lints errors and code cleanup

* Refactor Oauth settings

* Added new UI to manager client ids.

* Removed the clientIds params from EsriOAuth2.Initialize.  Make sure deleted entries are remove from setting service too. Fixed lint errors

* strings localization

* Enabled tests for ArcgisUtilities.

* Fixed display issues with Labelled issue and fix MapUrlDialog display issues when providing credentials password to an already attached layer.

* Fixed display issues with Labelled issue and fix MapUrlDialog display issues when providing credentials password to an already attached layer.

* CSS fine tuning.

* Fixed lint errors.

* Added change log and ajusted releases tags.

* Fix pre-commit hooks.

* Added missing changelog.

* Removed commented code.

* Close popup when an error occurs.

* Renamed usePopup for useCrossOriginPopup.
Moved useCrossOriginPopup under ui-core instead of ui-framework.
Fixed an issue in useCrossOriginPopup where popup would not get closed when the hook was unmounted.
Added tests for userInterval and useCrossOriginPopup .
Fixed typos.

* Setup an onClose listener on the popup's parent window, to close popup when the parent get closed.

* CSS fine tuning.

* Added change logs

* Added change log.

* Added missing @docs-group-description

* Added missing changelog

* override ui-framework.api.md to master version.

* Removed MapLayerAuthentification.ts (mow replaced by MapLayerAuthentication.ts)

* WIP - Isolate Oauth2 code under ArcGisAccessClient

* Moved classes to maplayers packages. Improved designed on ArcGisAccessClient.

* Made UI independent from ESRI Oauth concepts.

* WIP - Made UI independent from authentification implementation. Dropped EsriOauth2 static class. Improved URLs handling.
TODO: Tests.

* Moved authentication code under a new 'maplayers-auth' package.

* Renamed Esri* to ArcGis*. Create application configuration parameters for Oauth2 in ui-test-app.  Code clean up

* Code cleanup

* Dropped depreacated 'MapLayerAuthType'.  Restored MapUrlDialog tests. Code cleanup.

* Moved ArcGisUtilities tests to maplayers-auth package.

* Moved ArcGisTokenGenerator and ArcGisTokenManager to maplayers-auth packages,

* Improved ArcGisAccessClient.initialize design + documentation.

* Fixed lint errors.

* Allow any custom data to be passed to the authorize endpoint.  No longer save token to localStorage even though 'Kepp me signed in' has been checked by end-user.

* VSCode launch entry.

* Fixed minor ui issues when invalid credentials were provided.

* api extract

* api-extract

* Fine tuned release tags and fixed lint errors.

* pnpm-lock.yaml

* Removed testing code.

* change logs

* api-extract

* Fixed last lint error

* remove .only in tests.

* Improved core-react tests to reach 100% code coverage.

* fixed package documentation.

* Removed @bentley changelogs.

* Updated maplayers-auth README's title.

* Moved react to dependencies.

* Update pnpm-lock.yaml

* Updated NextVersion.md

* Update pnpm-lock.yaml

* Upgraded jsom version to 19.0.0 in map-layers-auth package.

* Added extract-api.  Fine tuned release tags.

* Removed whitespace changes to .vscode

* Promoted release tags from alpha to beta.

* More release tags adjustements.

* core-front api extract.

* core-frontend api extract.

* extract-api

Co-authored-by: Caleb Shafer <31107829+calebmshafer@users.noreply.github.com>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

* TransformElementsTool changes to support copy. (#3607)

* TransformElementsTool changes to support copy.

* Move disclaimer to CopyElementsTool class documentation.

* 3.2.0-dev.71

* initial support for bentley cra5 (#3419)

Co-authored-by: Arun George <aruniverse@users.noreply.github.com>

* refactor extension types and begin remote (runtime) extension support (#3347)

Co-authored-by: johnnyd710 <19596966+johnnyd710@users.noreply.github.com>

* @bentley/imodeljs-native 3.2.8

* extract-api

* lint

* fix not mapping codespecs and double checking jsonProperties in test developed before improvements to identity transform assert util

* Fixed WKT name that was not trimmed previously. Changed to a name that will work in both versions of native

(cherry picked from commit 803fc52)

* Rush change

(cherry picked from commit de72a84)

* lint

* remove references to non-existent snippets.

* 3.2.0-dev.72

* liiiiiiinnnttt......................

* dooooc liiinnkss....

Co-authored-by: Michael Belousov <mike.belousov@bentley.com>
Co-authored-by: Michael Belousov <MichaelBelousov@users.noreply.github.com>
Co-authored-by: NancyMcCallB <45079789+NancyMcCallB@users.noreply.github.com>
Co-authored-by: imodeljs-admin <38288322+imodeljs-admin@users.noreply.github.com>
Co-authored-by: Robert Lukasonok <70327485+roluk@users.noreply.github.com>
Co-authored-by: Caleb Shafer <31107829+calebmshafer@users.noreply.github.com>
Co-authored-by: Arun George <aruniverse@users.noreply.github.com>
Co-authored-by: johnnyd710 <19596966+johnnyd710@users.noreply.github.com>
Co-authored-by: Bill Steinbock <65047615+bsteinbk@users.noreply.github.com>
Co-authored-by: Mark Schlosser <47000437+markschlosseratbentley@users.noreply.github.com>
Co-authored-by: Michel D'Astous <mdastous-bentley@users.noreply.github.com>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
Co-authored-by: bbastings <65233531+bbastings@users.noreply.github.com>
Co-authored-by: Arun George <11051042+aruniverse@users.noreply.github.com>
Co-authored-by: kckst8 <6283674+kckst8@users.noreply.github.com>
Co-authored-by: AlainRobertAtBentley <73677355+AlainRobertAtBentley@users.noreply.github.com>
Co-authored-by: Alina Paliulionytė <67429235+Alina657@users.noreply.github.com>
Co-authored-by: Alina Paliulionyte <Alina657@users.noreply.github.com>
Co-authored-by: Daniel Toby <41296254+DanielToby@users.noreply.github.com>
…sent (#3642)

* move up equalWithFpTolerance and add test for placement geometry json roundtrip

* remove dead imports from development

* rush change

Co-authored-by: Michael Belousov <MichaelBelousov@users.noreply.github.com>
#3656)

* close transformer state dumps even if producing them threw an error

Co-authored-by: Michael Belousov <MichaelBelousov@users.noreply.github.com>

Co-authored-by: Michael Belousov <MichaelBelousov@users.noreply.github.com>
* work on cloud containers

* make workspace containers use CloudSqlite

* wip

* workspace editor with new CloudSqlite

* support CloudContainers for opening SQLiteDbs

* add tests for CloudSqlite

* test anonymous access

* add tests for using CloudCache guid for write locks

* work on download checkpoint

* more download testing

* allow creating and opening SQLiteDbs with params

* cloud workspace tests

* rework resolveContainerName

* add WorkspaceDb versioning

* fix tests

* lint errors

* enable checksums for block names

* add makeVersion command to WorkspaceEditor

* documentation cleanup

* documentation cleanup

* add --noPrompt for initializeWorkspace

* support "@" scripts

* fix package file

* cleanup

* don't allow writes to workspaceDbs unless they have been versioned in the current txn

* fix tests

* rename container.attach to container.connect

* fix tests

* add detach arg to disconnect

* separate detach and disconnect

* allow offline usage

* collate nocase for listDb

* add detach command to WorkpsaceEditor

* show "friendly" file sizes

* add version for testing cloud gcs

* renames for customuri api

* workspace settings

* checkForChanges is not async

* get tests working

* load extra gcs databases stored in iModel settings

* work on loading settings

* work on settings

* read all settings and schema files at startup

* fix tests

* old->newV2props, add test for daemon, start working on supporting v2checkpoint without daemon

* doc cleanup

* don't rename V2CheckpointAccessProps members

* fixes, Nick please review

* doc cleanup

* wip

* merge

* fix tests

* cleanup lint warnings

* Add some tests, more changes around cache / blockcache_dir

* doc

* Add two env variables, prefetchbeforeopen, prefetchafteropen for testing. add non null assertions to some failing tests

* wip

* lint errors

* fix tests

* validate settings against schema

* validate array entries

* validate setting schemas

* workspace documentation

* afds

* rename curl debugging option

* wip

* time prefetch

* make editor functions async

* workspaceEditor doc

* document @ scripts

* workspaceEditor doc

* @ script examples

* cleanup

* new sqlite code for prefetch

* prefetch testing

* cleanup

* add a PREFETCH_TIMe env variable

* make WorkspaceDb.prefetch return the prefetch object

* wip

* merge

* update @types/chai

* add prefetch_nrequests

* use parseSync for Yargs

* log time taken for prefetch always

* lint errors

* allow choosing nRequests for prefetch testing

* don't use env variables, switch to app settings to determine a prefetch

* improve prefetch logging

* lint errors

* cleanup

* allow importDb to supply extension for input file

* use CloudContainer.blockSize

* lint errors

* use https for gcs data url

Co-authored-by: nick.tessier <22119573+nick4598@users.noreply.github.com>
* Functionality to generate sha1 hash for schema xml

* consuming iModelHost

* Adding static function to the schema class

* Made the hash generation as separate functionality due to circular import issue

* removed unnecessary import

* function name updated with lint fixes

* Moving changes to IModelHost

* Documentation removed

* combined test

* line space added back

* changelogs added
# Conflicts:
#	common/changes/@bentley/imodelbank-client/3.2.1-changelogs_2022-06-07-15-15.json
#	common/changes/@bentley/imodelbank-client/3.2.2-changelogs_2022-06-10-16-17.json
#	common/config/rush/pnpm-lock.yaml
#	core/backend/package.json
#	core/backend/src/HubMock.ts
#	core/backend/src/LocalHub.ts
#	core/backend/src/test/hubaccess/CheckpointManager.test.ts
#	core/backend/src/test/index.ts
#	core/backend/src/test/standalone/TileCache.test.ts
#	core/bentley/package.json
#	core/common/package.json
#	core/transformer/package.json
#	core/transformer/src/test/IModelTransformerUtils.ts
#	core/transformer/src/test/standalone/IModelTransformerHub.test.ts
#	core/transformer/src/test/standalone/IModelTransformerResumption.test.ts
#	domains/analytical/backend/package.json
#	editor/common/package.json
#	extensions/map-layers-auth/package.json
#	full-stack-tests/rpc/package.json
#	test-apps/export-gltf/package.json
#	test-apps/imjs-importer/package.json
#	test-apps/imodel-from-geojson/package.json
#	test-apps/imodel-from-orbitgt/package.json
#	test-apps/imodel-from-reality-model/package.json
#	test-apps/imodel-transformer/package.json
@pmconne
Copy link
Member Author

pmconne commented Jun 20, 2022

@mdastous-bentley see how you like it now.

@pmconne
Copy link
Member Author

pmconne commented Jun 20, 2022

@grigasp @saskliutas @roluk presentation test timing out fyi.

  1) Ruleset Variables
       Multiple backends for one frontend
         "before each" hook for "can use the same frontend-registered ruleset variables after backend is reset":
     Error: Timeout of 60000ms exceeded.

@pmconne
Copy link
Member Author

pmconne commented Jun 20, 2022

@calebmshafer please review pinned dependencies.

@calebmshafer
Copy link
Member

calebmshafer commented Jun 20, 2022

@calebmshafer please review pinned dependencies.

@pmconne just saw this, reviewing now.

* download a v2 checkpoint
* @internal
*/
downloadV2Checkpoint: (arg: CheckpointArg) => Promise<ChangesetIndexAndId>;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this is going to require a change in the client libs for the iModels API. FYI @austeja-bentley @wgoehrig

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hopefully not. It was a mistake for it to have ever been in this interface.

@MichaelBelousov
Copy link
Contributor

MichaelBelousov commented Jun 20, 2022

Looks like the merge reintroduced a pretest package.json script from #2430 that isn't in master.
This appears to trigger a windows-specific bug in nyc which I've reported here.
The pretest script isn't necessary and I'm going to push a commit removing it shortly.

@pmconne
Copy link
Member Author

pmconne commented Jun 21, 2022

@kabentley may the GeoCoord errors be causing some tests to time out after 60 seconds?

2022-06-20T22:48:45.9724507Z     Using variables in rules
2022-06-20T22:48:45.9725629Z Error | GeoCoord | Error: attach error: Timeout was reached, account=https://gcs-data.itwin.org/ {"ActivityId":"1d80902b-1d74-43c6-ae72-2f08088914dd","SessionId":"4739dc65-a4ee-43b7-8f93-92cefbbc63ae","ApplicationId":"2686","ApplicationVersion":"1.0.0","rpcMethod":"openFile"}
2022-06-20T22:48:45.9726514Z       1) "before each" hook for "can specify lots of ids with Id64[] overload"
2022-06-20T23:00:47.2140261Z   1) Ruleset Variables
2022-06-20T23:00:47.2140787Z        Using variables in rules
2022-06-20T23:00:47.2141334Z          "before each" hook for "can specify lots of ids with Id64[] overload":
2022-06-20T23:00:47.2143162Z      Error: Timeout of 60000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves. (/agent/_work/1/s/full-stack-tests/presentation/lib/frontend/RulesetVariables.test.js)
2022-06-20T23:00:47.2144143Z   
2022-06-20T23:00:47.2144497Z 
2022-06-20T23:00:47.2144972Z   2) Learning Snippets
2022-06-20T23:00:47.2145467Z        "before each" hook for "using for customization":
2022-06-20T23:00:47.2146931Z      Error: Timeout of 60000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves. (/agent/_work/1/s/full-stack-tests/presentation/lib/learning-snippets/common/RelatedInstanceSpecification.test.js)
2022-06-20T23:00:47.2148045Z       at processTicksAndRejections (node:internal/process/task_queues:96:5)
2022-06-20T23:00:47.2148435Z 
2022-06-20T23:00:47.2148806Z   3) Learning Snippets
2022-06-20T23:00:47.2149328Z        "before each" hook for "uses `handleTargetClassPolymorphically` attribute":
2022-06-20T23:00:47.2150764Z      Error: Timeout of 60000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves. (/agent/_work/1/s/full-stack-tests/presentation/lib/learning-snippets/content/customization/RelatedPropertiesSpecification.test.js)
2022-06-20T23:00:47.2151740Z       at runMicrotasks (<anonymous>)
2022-06-20T23:00:47.2152324Z       at processTicksAndRejections (node:internal/process/task_queues:96:5)
2022-06-20T23:00:47.2152726Z 
2022-06-20T23:00:47.2153089Z   4) Learning Snippets
2022-06-20T23:00:47.2153489Z        Content
2022-06-20T23:00:47.2153937Z          "before each" hook for "uses `label` attribute":
2022-06-20T23:00:47.2155201Z      Error: Timeout of 60000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves. (/agent/_work/1/s/full-stack-tests/presentation/lib/learning-snippets/ContentCustomization.test.js)
2022-06-20T23:00:47.2156091Z       at runMicrotasks (<anonymous>)
2022-06-20T23:00:47.2156671Z       at processTicksAndRejections (node:internal/process/task_queues:96:5)
2022-06-20T23:00:47.2157055Z 
2022-06-20T23:00:47.2157444Z   5) Learning Snippets
2022-06-20T23:00:47.2157946Z        "before each" hook for "uses `class` attribute":
2022-06-20T23:00:47.2159252Z      Error: Timeout of 60000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves. (/agent/_work/1/s/full-stack-tests/presentation/lib/learning-snippets/customization/DisabledSortingRule.test.js)
2022-06-20T23:00:47.2160164Z       at runMicrotasks (<anonymous>)
2022-06-20T23:00:47.2160731Z       at processTicksAndRejections (node:internal/process/task_queues:96:5)
2022-06-20T23:00:47.2161125Z 
2022-06-20T23:00:47.2161488Z   6) Learning Snippets
2022-06-20T23:00:47.2161979Z        "before each" hook for "uses `hideIfNoChildren` attribute":
2022-06-20T23:00:47.2163311Z      Error: Timeout of 60000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves. (/agent/_work/1/s/full-stack-tests/presentation/lib/learning-snippets/hierarchies/specifications/Shared.test.js)
2022-06-20T23:00:47.2164470Z       at runMicrotasks (<anonymous>)
2022-06-20T23:00:47.2165040Z       at processTicksAndRejections (node:internal/process/task_queues:96:5)

@pmconne
Copy link
Member Author

pmconne commented Jun 21, 2022

backend-integration-tests are failing then terminating abnormally. I tried pinning azurite to the version (3.14.0) used on master to no effect.

  Checkpoints
INFO(e) [2022/06/20 22:35:19.632]: listening on localhost port 2030
    1) should use fallback directory when blockcache_dir has no daemon in it
    2) "after each" hook for "should use fallback directory when blockcache_dir has no daemon in it"
    3) "after all" hook for "should be able to open and read V2 checkpoint with daemon running"

  Cloud workspace containers
    4) cloud workspace

  IModelOpen
    ✔ Unauthorized requests should cause an obvious error
    5) should be able to handle simultaneous open calls
terminate called after throwing an instance of 'Napi::Error'
  what():  Error in native callback
Aborted (core dumped)
Azurite Blob service is closing...
Azurite Blob service successfully closed
ERROR: "test:integration:core" exited with 134.
ERROR: "test:integration:azurite" exited with 1.

@kabentley
Copy link
Contributor

backend-integration-tests are failing then terminating abnormally. I tried pinning azurite to the version (3.14.0) used on master to no effect.

I will have to talk to @nick4598. Something about starting/stopping the daemon isn't right. I skipped those tests for now.

@pmconne pmconne merged commit 8735b84 into master Jun 21, 2022
@pmconne pmconne deleted the native/3.3.3 branch June 21, 2022 12:56
@pmconne pmconne mentioned this pull request Jul 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet