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

Generate sub created events only if recursive=True #454

Merged

Conversation

rrzaripov
Copy link
Contributor

When directory with some files will be copied in to observed directory, watchdog emits events about that files created, even if recursive switched off.

@danilobellini
Copy link
Collaborator

You're right! Can you add some test cases?

I think a case that's worth testing is a directory being renamed just after being added, after the WATCHDOG_TRAVERSE_MOVED_DIR_DELAY hack. The directory renaming event should be emitted, but any new file added to the the directory (before and after it had been renamed) shouldn't (that's the issue you're talking about). Also, as that should behave the same on every environment/OS, a test on that would be a great enhancement to this project. =)

@BoboTiG
Copy link
Collaborator

BoboTiG commented Sep 18, 2018

@rrzaripov could you open a specific PR for AppVeyor please?

@rrzaripov
Copy link
Contributor Author

@BoboTiG yes, of course, changes pushed to this branch by my mistake. I'm will notify when fix this PR and tests will be added.

@BoboTiG
Copy link
Collaborator

BoboTiG commented Oct 13, 2018

Could you rebase your commit on master and add a test for that change please? It would be great :)

@rrzaripov
Copy link
Contributor Author

@BoboTiG sorry for long awaiting for update PR. Well I'm write basic tests for check how the watchdog working with and without recursive. Under Windows tests is more like lottery, fails once, success in another run. I'm still work for making tests more stability. Any help, advices are welcome!

@BoboTiG
Copy link
Collaborator

BoboTiG commented Dec 20, 2018

You should first rebase your work from the master branch as merging is blocked right now :/
I will give you some help when I can :)

When directory with some files will be copied in to observed directory, watchdog emits events about that files created, even if recursive switched off.

Add Windows support in emitter tests, fix some PEP-8 issues.

Add Empty for handling exceptions in tests
@rrzaripov rrzaripov force-pushed the fix-recursive-issue-under-windows branch from 75fff00 to e1ea356 Compare December 20, 2018 19:47
@rrzaripov
Copy link
Contributor Author

@BoboTiG rebase done:)

@BoboTiG BoboTiG merged commit b8cf1b0 into gorakhargosh:master Feb 16, 2019
@BoboTiG
Copy link
Collaborator

BoboTiG commented Feb 16, 2019

Thanks @rrzaripov :)

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.

None yet

3 participants