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

Module not found error in all the tests. #12

Open
debojitkakoti opened this issue Jun 7, 2016 · 7 comments
Open

Module not found error in all the tests. #12

debojitkakoti opened this issue Jun 7, 2016 · 7 comments

Comments

@debojitkakoti
Copy link

ERROR in ./test/routers/test_logged_out.jsx Module not found: Error: Cannot resolve 'file' or 'directory' ../../src/routers/logged_out in /home/workspace/sb-admin-react/test/routers @ ./test/routers/test_logged_out.jsx 15:28-67
Steps to reproduce:
npm test

The import path in all the test cases is pointing to wrong directory !

@sanketsahu
Copy link
Contributor

Thanks for the report. We are not free at the moment. Can you fix that and send pull request?

@debojitkakoti
Copy link
Author

I will try to fix it. Thanks .

@ssteinerx
Copy link

None of the included files that are supposed to be in ../../src/common exist and it's unclear what's supposed to be in them. I looked back through the history and it doesn't seem that those files ever existed or that the test ever actually ran. NOTE: this is based on what's in the repository, I'm sure those files exist on someone's development machine, but they never made it into this repo.

@ssteinerx
Copy link

@sarika01 Do you have this files? I'm working on a fork and would love to start with some working tests, but the required files didn't get checked into this repo though I assume they existed on someone's development machine at some point.

@sarika01
Copy link
Contributor

@ssteinerx , those files were long back changed and renamed to the current structure. After that I haven't had the opportunity to write and update the test cases. But, just in case if it's helpful to you, I am attaching the files.
extra files.zip

@ssteinerx
Copy link

@sarika01 Thanks! I'll work them in and see if I can get them working with the current build. If they're too out-of-date, I'll probably just start over with new, comprehensive set of tests, possibly even using a different set of tools.

If you're interested, please see my fork.

I've updated all of the dependencies other than Babel (which requires a major restructuring of the config file), but have no unit tests at the moment to prove it's all right.

So far, by rearranging devDependencies vs dependencies, I've reduced reduced build/ contents from 3.9M in the original v0.3.0 to ~1.9M in my 0.4.1 fork.

If you're interested, you could cherry-pick my changes back into your repository as a base, and I could rebase to that as a starting point, then pull my changes back in in a series of small, atomic pull requests.

Otherwise, if you'd rather not continue to maintain this, I'd be happy to take over development. If so, you could just redirect to my fork, and we can figure out how to get this published to NPM. Obviously, I'd leave all original attribution to you, and link to your company site etc. if you'd like and will get your approval before publishing.

Please let me know as, once I have a solid set of tests in place, this will be moving pretty quickly as I'm integrating it into a paid client project and so will have quite a bit of time to work on it.

Thanks,

Steve Steiner
aka/ssteinerX

@sarika01
Copy link
Contributor

Hi @ssteinerx, it's really nice to see your active interest in this repo. We are going to maintain this repo in the long-term. Your changes and updated libraries are a great contribution and I would be happy to merge them here.
We would also be happy to get the NPM module published and your support for this would always be appreciated!
regards,
Sarika

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

No branches or pull requests

4 participants