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

Add name for factory intermediate input #3786

Closed
wants to merge 1 commit into from

Conversation

emrojo
Copy link
Contributor

@emrojo emrojo commented Apr 13, 2023

Use class instead of factory

Generate new name

Test with new

With new (should fail)

With create

Using factory

Add a random string to the factory

Print errors, maybe that will give more clues about the problem

Validating (hum...)

Test the purpose do not have errors before any action

Try with state

Try with alphabetic order

Rename alphabetical

Closes #

Changes proposed in this pull request:

  • ...

Use class instead of factory

Generate new name

Test with new

With new (should fail)

With create

Using factory

Add a random string to the factory

Print errors, maybe that will give more clues about the problem

Validating (hum...)

Test the purpose do not have errors before any action

Try with state

Try with alphabetic order

Rename alphabetical
@emrojo emrojo closed this Apr 13, 2023
@emrojo emrojo deleted the test_bugfix_factory_name branch April 13, 2023 14:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant