Set both Symbol.species and "@@species" on Concast constructor (#7403)

An attempt to address the problems I speculated about in
#6520 (comment)

Theory: in the Hermes JS engine, Symbol.species is not defined, so
Object.defineProperty was not called, but perhaps "@@species" was getting
set somewhere else by a polyfill library, causing the zen-observable
library to fall back to "@@species" instead of using/ignoring the
nonexistent Symbol.species symbol.

src/utilities/observables/Concast.ts

@@ -243,11 +243,19 @@ export class Concast<T> extends Observable<T> {
 // Those methods assume (perhaps unwisely?) that they can call the
 // subtype's constructor with an observer registration function, but the
 // Concast constructor uses a different signature. Defining this
-// Symbol.species getter function on the Concast constructor function is
-// a hint to generic Observable code to use the default constructor
-// instead of trying to do `new Concast(observer => ...)`.
+// Symbol.species property on the Concast constructor function is a hint
+// to generic Observable code to use the default constructor instead of
+// trying to do `new Concast(observer => ...)`.
+function setSpecies(key: symbol | string) {
+  // Object.defineProperty is necessary because Concast[Symbol.species]
+  // is a getter by default in modern JS environments, so we can't
+  // assign to it with a normal assignment expression.
+  Object.defineProperty(Concast, key, { value: Observable });
+}
 if (typeof Symbol === "function" && Symbol.species) {
-  Object.defineProperty(Concast, Symbol.species, {
-    value: Observable,
-  });
+  setSpecies(Symbol.species);
 }
+// The "@@species" string is used as a fake Symbol.species value in some
+// polyfill systems (including the SymbolSpecies variable used by
+// zen-observable), so we should set it as well, to be safe.
+setSpecies("@@species");