Remove Embroider workarounds in test embroider-basic-compat

[mkszepp]

In EmberData there are some workarounds for embroider. Cause of this workaround the ember-data embroider tests were not failed in v4.12.x and this has produced this side-effect embroider-build/embroider#1506 (comment) in consumer app.

For EmberData v5 there makes no different, but ember data v4.12.x (maybe also 4.11.x) there is necessary to add workaround in consumer app to get run ember-data.

data/tests/embroider-basic-compat/ember-cli-build.js

Lines 35 to 55 in 96cad26

	compatAdapters: new Map([
	['@ember-data/store', null],
	['@ember-data/record-data', null],
	['@ember-data/serializer', null],
	['@ember-data/adapter', null],
	['@ember-data/model', null],
	['@ember-data/debug', null],
	['@ember-data/tracking', null],
	['@ember-data/request', null],
	['ember-data', null],
	]),
	packageRules: [
	{
	package: '@ember-data/store',
	addonModules: {
	'-private.js': {
	dependsOnModules: ['@ember-data/json-api'],
	},
	},
	},
	],

We should remove this lines for ember-data v5 & after embroider release also for ember-data v4

[runspired]

These lines are here because we are not compatible with embroider if embroider is secretly trying to fix things. Embroider should never be providing a compat adapter 4.12+. Even once embroider has removed them I think we should keep this just in case, I don't want to get trolled thinking there's support where there isn't.