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 migrations folder to execute dumpdata. #86

Conversation

fredericosachweh
Copy link

I tried execute dumpdata but I haven't the tables in my database and I created a migration to fix my problem.

@gustavi
Copy link

gustavi commented Jan 10, 2016

Because of inactivity of the project and no answer of the author our team decide to fork it. Please create a pull request if this is always required.

@adamchainz
Copy link
Owner

Hi, I'm keen to merge this and fix the problem, but I'm having a trouble replicating it in tests. Adding this test to the test suite passes:

class MigrationsTests(TestCase):

    def test_check_dumpdata_works(self):
        CorsModel.objects.create(cors='x')
        call_command('dumpdata')

Is your issue that the table for CorsModel didn't exist for you? It seems to be auto-created in tests here.

Anyway I'll be looking at making the model optional in #121 since I think most peoples' use cases don't need it.

@adamchainz
Copy link
Owner

#153 will invalidate this by removing the concrete model entirely 😄

@adamchainz adamchainz closed this Oct 20, 2016
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.

3 participants