@firebase/rules-unit-testing fails when loaded as es6 module with firebase-js-sdk >= 9.2.0

[kenkunz]

[REQUIRED] Describe your environment

• Operating System version: macOS Monterey 12.0.1
• Browser version: n/a
• Firebase SDK version: >= 9.2.0
• Firebase Product: rules-unit-testing, firestore

[REQUIRED] Describe the problem

With @firebase/rules-unit-testing@2.0.1 and firebase >=9.0.0 <=9.1.3, the test utilities work
whether loaded as a cjs require or an es6 module. After upgrading to firebase >=9.2.0, the
utilities no longer work when loaded as es6 module, but still work when loaded with cjs require.

Since other firebase modules support es6 module loading, it seems surprising for rules-unit-testing
not to fully support this as well.

Current workaround

Using cjs require for my firestore rules tests.

Steps to reproduce:

1. install dependency: Firebase Emulators

2. clone kenkunz/firestore-testing-repro and install npm modules

   git clone https://github.com/kenkunz/firestore-testing-repro.git
   cd firestore-testing-repro
   npm install

3. run the tests

   npm run test

   Note that both test.js and test.cjs complete successfully

4. upgrade to firebase@9.2.0 (or greater)

   npm i -D firebase@9.2.0

5. run the tests again

   npm run test

   Note that test.cjs still passes but test.js fails (see error output below)

Error output:

FirebaseError: Expected first argument to collection() to be a CollectionReference, a DocumentReference or FirebaseFirestore
at doc (file:///Users/ken/Code/firestore-testing-repro/node_modules/@firebase/firestore/dist/index.node.mjs:19998:19)
at file:///Users/ken/Code/firestore-testing-repro/test.js:8:20
at RulesTestEnvironmentImpl.<anonymous> (node_modules/@firebase/rules-unit-testing/dist/index.cjs.js:310:46)
at step (node_modules/@firebase/rules-unit-testing/dist/index.cjs.js:74:23)
at Object.next (node_modules/@firebase/rules-unit-testing/dist/index.cjs.js:55:53)
at /Users/ken/Code/firestore-testing-repro/node_modules/@firebase/rules-unit-testing/dist/index.cjs.js:48:71
at new Promise (<anonymous>)
at __awaiter (node_modules/@firebase/rules-unit-testing/dist/index.cjs.js:44:12)
at RulesTestEnvironmentImpl.withSecurityRulesDisabled (node_modules/@firebase/rules-unit-testing/dist/index.cjs.js:300:16)
at Context.<anonymous> (file:///Users/ken/Code/firestore-testing-repro/test.js:7:19)
at processTicksAndRejections (node:internal/process/task_queues:96:5)

[google-oss-bot]

I couldn't figure out how to label this issue, so I've labeled it for a human to triage. Hang tight.

[jbalidiong]

Hi @kenkunz, thanks for the report. I was able to reproduce the behavior now. Let me check what we can do for this issue or bring someone here that can provide more context about it. I’ll update this thread if I have any information to share.

[angrycat9000]

Thanks for the work around @kenkunz. I hit this bug when upgrading to the new 9.0 API

[ChromeQ]

Noticed this is old but it is still an issue with firebase v9.9.3 and @firebase/rules-unit-testing v2.0.4 so it would be good to see an update or a fix here

[ghinda]

Seems to be fixed with with the latest firebase 11.0.1 and @firebase/rules-unit-testing 4.0.0.

[DellaBitta]

@kenkunz,

Can you confirm that this was fixed in v11?

[kenkunz]

@kenkunz,

Can you confirm that this was fixed in v11?

Sorry, I haven't been actively using Firebase for a while, and don't have bandwidth to dust this off to test atm.