search

themsaid / laravel-langman

Language files manager in your artisan console.
MIT License
876 stars 166 forks source link

all tests is broken #25

Closed ahmedash95 closed 8 years ago

ahmedash95 commented 8 years ago

i'm trying to solve the problem but i think you could do it faster than me , could you please check all tests

@themsaid

themsaid commented 8 years ago

All tests where?

themsaid commented 8 years ago

If you mean the PR then let's discuss it there.

ahmedash95 commented 8 years ago

@themsaid i didn't mean the PR . could you please try to run test with a fresh install of laravel with langman ?

themsaid commented 8 years ago

How do you run tests after pulling langman into laravel?

ahmedash95 commented 8 years ago

cd vendor/themsaid/laravel-langman

then

composer install

then

phpunit

themsaid commented 8 years ago

Can you share the errors coming from running the tests?

ahmedash95 commented 8 years ago

http://i.imgur.com/mAEMvdS.png

themsaid commented 8 years ago

I think the script isn't able to create the temporary test files, anyway it's recommended that you develop the package outside laravel.

ahmedash95 commented 8 years ago

how can i test my modifications if i developing the package outside laravel ?

themsaid commented 8 years ago

That's the point of testing :) Your tests need to be isolated. Anyway you should track teh reason behind the failing tests on your system, it's not a general issue as the tests pass on Travis: https://travis-ci.org/themsaid/laravel-langman

ahmedash95 commented 8 years ago

really weird , i will try to fix this issue , thanks for your time.