[tc39/ecma262#1645] Remove double error tests (#2290)

Due to changing idempotenxy for dynamic import on failures these tests are no longer needed.
After tc39/ecma262#1645 idempotency is only required after completing normally.

test/language/expressions/dynamic-import/double-error-resolution-promise.js

@@ -1,47 +0,0 @@
-// Copyright (C) 2018 Leo Balter. All rights reserved.
-// This code is governed by the BSD license found in the LICENSE file.
-/*---
-description: >
-    Re-resolve a poisoned module should be consistent on the failure path
-esid: sec-finishdynamicimport
-info: |
-    Runtime Semantics: HostImportModuleDynamically
-
-        Failure path
-            - At some future time, the host environment must perform FinishDynamicImport(referencingScriptOrModule,
-            specifier, promiseCapability, an abrupt completion), with the abrupt completion representing the cause
-            of failure.
-
-        The intent of this specification is to not violate run to completion semantics. The spec-level formalization
-        of this is a work-in-progress.
-
-        Every call to HostImportModuleDynamically with the same referencingScriptOrModule and specifier arguments
-        must conform to the same set of requirements above as previous calls do. That is, if the host environment
-        takes the success path once for a given referencingScriptOrModule, specifier pair, it must always do so,
-        and the same for the failure path.
-
-    Runtime Semantics: FinishDynamicImport ( referencingScriptOrModule, specifier, promiseCapability, completion )
-
-    2. Otherwise,
-        a. Assert: completion is a normal completion and completion.[[Value]] is undefined.
-        b. Let moduleRecord be ! HostResolveImportedModule(referencingScriptOrModule, specifier).
-        c. Assert: Evaluate has already been invoked on moduleRecord and successfully completed.
-        d. Let namespace be GetModuleNamespace(moduleRecord).
-        ...
-        f. Otherwise, perform ! Call(promiseCapability.[[Resolve]], undefined, « namespace.[[Value]] »).
-flags: [async]
-features: [dynamic-import]
-includes: [compareArray.js]
----*/
-
-function callImport(name) {
-    return import('./double-error-resolution_FIXTURE.js').catch(error => {
-        assert.sameValue(error, 'foo');
-        return `caught ${name}`;
-    });
-}
-
-Promise.all([
-    callImport(1),
-    callImport(2)
-]).then(resolutions => assert.compareArray(resolutions, ['caught 1', 'caught 2'])).then($DONE, $DONE);

test/language/expressions/dynamic-import/double-error-resolution.js

@@ -1,60 +0,0 @@
-// Copyright (C) 2018 Leo Balter. All rights reserved.
-// This code is governed by the BSD license found in the LICENSE file.
-/*---
-description: >
-    Re-resolve a poisoned module should be consistent on the failure path
-esid: sec-finishdynamicimport
-info: |
-    Runtime Semantics: HostImportModuleDynamically
-
-        Failure path
-            - At some future time, the host environment must perform FinishDynamicImport(referencingScriptOrModule,
-            specifier, promiseCapability, an abrupt completion), with the abrupt completion representing the cause
-            of failure.
-
-        The intent of this specification is to not violate run to completion semantics. The spec-level formalization
-        of this is a work-in-progress.
-
-        Every call to HostImportModuleDynamically with the same referencingScriptOrModule and specifier arguments
-        must conform to the same set of requirements above as previous calls do. That is, if the host environment
-        takes the success path once for a given referencingScriptOrModule, specifier pair, it must always do so,
-        and the same for the failure path.
-
-    Runtime Semantics: FinishDynamicImport ( referencingScriptOrModule, specifier, promiseCapability, completion )
-
-    2. Otherwise,
-        a. Assert: completion is a normal completion and completion.[[Value]] is undefined.
-        b. Let moduleRecord be ! HostResolveImportedModule(referencingScriptOrModule, specifier).
-        c. Assert: Evaluate has already been invoked on moduleRecord and successfully completed.
-        d. Let namespace be GetModuleNamespace(moduleRecord).
-        ...
-        f. Otherwise, perform ! Call(promiseCapability.[[Resolve]], undefined, « namespace.[[Value]] »).
-flags: [async]
-features: [dynamic-import]
----*/
-
-async function fn() {
-    let err;
-    let result = Object.create(null);
-    const keep = result;
-    try {
-        result = await import('./double-error-resolution_FIXTURE.js');
-    } catch (error) {
-        err = error;
-    }
-    assert.sameValue(err, 'foo', 'first evaluation should be an abrupt completion');
-    assert.sameValue(result, keep, 'result should not be set');
-
-    err = undefined;
-
-    try {
-        result = await import('./double-error-resolution_FIXTURE.js');
-    } catch (error) {
-        err = error;
-    }
-
-    assert.sameValue(result, keep, 'result should still be the same as keep');
-    assert.sameValue(err, 'foo', 'second evaluation should repeat the same abrupt completion');
-}
-
-fn().then($DONE, $DONE);

test/language/expressions/dynamic-import/double-error-resolution_FIXTURE.js

@@ -1,8 +0,0 @@
-// Copyright (C) 2018 Leo Balter. All rights reserved.
-// This code is governed by the BSD license found in the LICENSE file.
-
-export var x = 42;
-
-throw 'foo';
-
-export var y = 1612;