-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Exit with a special code for tool/browser failure #1976
Comments
Here is a list of potential errors and suggested exit codes to align with protractor for consistent error reporting: Error code / Error category |
Where would you suggest to change/add these exit codes? I have seen this myself and it is indeed, as you say, not useful to use one exit code for all failures/errors. I would love to work on this, but I would like to know the correct and most useful place to set the exit codes. If it is possible to help me figure this out, I'll start working on it! |
If we do this I would like to adhere to standard exit codes defined in #define EX_OK 0 /* successful termination */
#define EX__BASE 64 /* base value for error messages */
#define EX_USAGE 64 /* command line usage error */
#define EX_DATAERR 65 /* data format error */
#define EX_NOINPUT 66 /* cannot open input */
#define EX_NOUSER 67 /* addressee unknown */
#define EX_NOHOST 68 /* host name unknown */
#define EX_UNAVAILABLE 69 /* service unavailable */
#define EX_SOFTWARE 70 /* internal software error */
#define EX_OSERR 71 /* system error (e.g., can't fork) */
#define EX_OSFILE 72 /* critical OS file missing */
#define EX_CANTCREAT 73 /* can't create (user) output file */
#define EX_IOERR 74 /* input/output error */
#define EX_TEMPFAIL 75 /* temp failure; user is invited to retry */
#define EX_PROTOCOL 76 /* remote error in protocol */
#define EX_NOPERM 77 /* permission denied */
#define EX_CONFIG 78 /* configuration error */
#define EX__MAX 78 /* maximum listed value */
#endif /* !_SYSEXITS_H_ */
We should not be using
|
@Moumi thanks for the offer but this is quite intricate and will involve a good amount of changes inside the code base so I'm not ready to commit to say we are doing this. Please take a look at the |
Expected behavior
If karma tests fail due to browser issue karma single run should exit with an exit code to differentiate between a 'tool failure' vs. a 'test failure'.
Would like karma to 'exit(100)' to indicate this type of failure or the error code must be configurable for these situations.
Actual behavior
Karma always exits with a '1' status whenever anythinng goes wrong, test failure or browser failure
Enviroment Details
karma --version
):karma.config.js
fileThe text was updated successfully, but these errors were encountered: