Skip to content

About fixtures directory set problem#24

Open
yaoelvon wants to merge 6 commits into
croach:masterfrom
yaoelvon:master
Open

About fixtures directory set problem#24
yaoelvon wants to merge 6 commits into
croach:masterfrom
yaoelvon:master

Conversation

@yaoelvon

Copy link
Copy Markdown

When I set app.config['FIXTURES_DIRS'], FIXTURES_DIRS path and "current_app.root_path + '/fixtures'" together have a same file 'author.json', flask-fixtures will get "current_app.root_path + '/fixtures/author.json'" file.
I think that default can use "current_app.root_path + '/fixtures'", but we shoud use app.config['FIXTURES_DIRS'] when I had set "app.config['FIXTURES_DIRS']".

@yaoelvon

Copy link
Copy Markdown
Author

Why CI build failed? I run 'nosetests' local successfully.

➜  Flask-Fixtures git:(master) nosetests
/Library/Python/2.7/site-packages/flask_sqlalchemy/__init__.py:800: UserWarning: SQLALCHEMY_TRACK_MODIFICATIONS adds significant overhead and will be disabled by default in the future.  Set it to True to suppress this warning.
  warnings.warn('SQLALCHEMY_TRACK_MODIFICATIONS adds significant overhead and will be disabled by default in the future.  Set it to True to suppress this warning.')
..........
----------------------------------------------------------------------
Ran 10 tests in 0.296s

OK

And I have do following:

    def tearDown(self):
        app.config.pop('FIXTURES_DIRS')
        self.app_context.pop()

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