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

Remove autostart #2374

Open
cah-brian-gantzler opened this issue Mar 23, 2022 · 7 comments
Open

Remove autostart #2374

cah-brian-gantzler opened this issue Mar 23, 2022 · 7 comments

Comments

@cah-brian-gantzler
Copy link
Collaborator

Is anyone actually using the autostart environment variable? I would assume this starts the server in the tests instead of using setupMirage. I can find it documented and using setupMirage is the preferred way to start the server.

Are we able to remove it since it is not documented? Would be nice to remove an instance-initializer if its not being used.

References:
https://github.com/miragejs/ember-cli-mirage/blob/master/addon/instance-initializers/ember-cli-mirage-autostart.js

@SergeAstapov
Copy link
Collaborator

@cah-briangantzler doesn't it make server running in development automatically?

@cah-brian-gantzler
Copy link
Collaborator Author

No, that is the enabled property https://www.ember-cli-mirage.com/docs/advanced/environment-options#enabled although autostart would have made more sense

@cah-brian-gantzler
Copy link
Collaborator Author

cah-brian-gantzler commented Mar 23, 2022

In the future I would propose that the enabled property also be deprecated and removed in favor of calling startMirage in the app. It is more explicit, and if your does not import and call startMirage embroider should be able to tree shake mirage out of ember s and have it only avail in ember t. Does embroider treat dev and test differently?

@cah-brian-gantzler
Copy link
Collaborator Author

Out of curiosity, I tried ember s and do not see @ember/test-helpers in the browser, given that autostart calls getRfc232TestContext(); and that checks the existence of @ember/test-helpers in https://github.com/miragejs/ember-cli-mirage/blob/master/addon/get-rfc232-test-context.js#L13. I am thinking it is only applying to tests

@jaswilli
Copy link
Contributor

jaswilli commented Apr 3, 2022

It seems pretty unlikely that a significant number of people are using autostart since it's not documented anywhere. I stumbled across it while troubleshooting some Ember 3.28 upgrade issues and even after reading through the discussion it wasn't clear to me whether or not it had parity with setupMirage() or if there was some non-obvious benefit to using it.

For the sake of simplicity I think removing it is a reasonable idea.

@bartocc
Copy link
Contributor

bartocc commented Nov 4, 2022

Our codebase is quite old and uses autostart. Some old doc can be found here https://github.com/miragejs/ember-cli-mirage/releases/tag/v0.4.2

@grapho
Copy link

grapho commented Apr 20, 2023

I am currently struggling with this as well.. even with a modern test setup (post 3.0) mirage server is still auto running somehow... causing setupMirage() to generate a 2nd Pretender deprecation..

Update: My ENV setup was wrong

if (environment === 'test') {
  ...
  ENV['ember-cli-mirage'] = {
    enabled: true,
  };

maybe this config was a relic from an older time pre-rfc-232/236?

but having this in ENV forces auto-run to happen despite this guard statement

maybe there needs to be some new (or restored 👀 ) documentation describing this behavior and how to properly migrate away from it?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants