6 kyu
Let's learn COBOL [Fix my noob mess] # 2
Loading description...
Fundamentals
Puzzles
This comment has been reported as {{ abuseKindText }}.
Show
This comment has been hidden. You can view it now .
This comment can not be viewed.
- |
- Reply
- Edit
- View Solution
- Expand 1 Reply Expand {{ comments?.length }} replies
- Collapse
- Spoiler
- Remove
- Remove comment & replies
- Report
{{ fetchSolutionsError }}
-
-
Your rendered github-flavored markdown will appear here.
-
Label this discussion...
-
No Label
Keep the comment unlabeled if none of the below applies.
-
Issue
Use the issue label when reporting problems with the kata.
Be sure to explain the problem clearly and include the steps to reproduce. -
Suggestion
Use the suggestion label if you have feedback on how this kata can be improved.
-
Question
Use the question label if you have questions and/or need help solving the kata.
Don't forget to mention the language you're using, and mark as having spoiler if you include your solution.
-
No Label
- Cancel
Commenting is not allowed on this discussion
You cannot view this solution
There is no solution to show
Please sign in or sign up to leave a comment.
target/generated_tests.cbl: 19: error: 'UT-TEST-CASE-NAME' is not defined target/generated_tests.cbl: 25: error: 'UT-TEST-CASE-COUNT' is not defined target/generated_tests.cbl: 26: error: 'UT-NORMAL-COMPARE' is not defined target/generated_tests.cbl: 27: error: 'UT-ACTUAL' is not defined target/generated_tests.cbl: 29: error: 'UT-EXPECTED' is not defined target/generated_tests.cbl: 30: error: 'UT-COMPARE-DEFAULT' is not defined target/generated_tests.cbl: 34: error: 'UT-TEST-CASE-NAME' is not defined target/generated_tests.cbl: 36: error: 'UT-TEST-CASE-COUNT' is not defined target/generated_tests.cbl: 37: error: 'UT-NORMAL-COMPARE' is not defined target/generated_tests.cbl: 38: error: 'UT-ACTUAL' is not defined target/generated_tests.cbl: 40: error: 'UT-EXPECTED' is not defined
Cant tell whats wrong. Is there an error with the test-cases? I cant tell what im missing, or if im to fix the test-cases. Dont know how to tbh since its exactly like https://www.codewars.com/kata/61618efa52747c000dd1347c/train/cobol testcases but wont take em here
I can confirm the tests are fine, you're probably missing something minor in your code like syntax or indentation.
Thanks Elfein7Night for answering, and yes its an error in your code, gl :) (I can see where the error is, looking at your code)
its pretty hard to spot, and you are not the first person that was caught by it
Alrighty thanks you two :) Will try again!
omg yupp just saw it. Weird error-feedback tho but well its cobol in the end xD Ty for the quick response!
Congrats on solving it :)
Ty!
Boop, Awaiting Approval.
I get a warning when I run your code inside the editor still:
I think it pops up because I switched to using
ut-expectedfromut-actual.It's merely logging it.
BAM! approved
Thanks :)
This comment has been hidden.
Yeah, not even 6 kyu yet
This comment has been hidden.
Am I pushing this too far? :p
edited.
I feel like an idiot for missing that
Na. its good to be able to improve my kata (if only the docs were that helpful)
Fixed.
Check and confirm :) (also hopefully approve)
solved
hurray :)
Hey LearningFTW, nice kata. I was thinking maybe
UT-ACTUALcould be renamed to somthing else maybeUT-EXPECTEDsince usually in tests theactualvalue is what the tested function returns. Not super critical, I know, but still :^)Sure, re-neamed
This comment has been hidden.
hahaha :D
This comment has been hidden.
looks good to me, approved.
Awaiting approval :)
Looks approvable to me. I hope B1ts, or any other COBOL veteran, would still have a look at it first.
ok :) will wait
Thanks :)
Random tests not ok: Please verify your random tests, they will always succeed, no matter which impl the user provides.
really ? gimme a minute
This comment has been hidden.
This comment has been hidden.
He who does nothing makes no mistakes
This comment has been hidden.
Updated
fixed
No random tests required for this one?
I though about it, but honestly if you wanted to hardcode it you need to know more about COBOl than this kata requires.
so didn't add them, if you think I should, I can add them ?
This comment has been hidden.
ok, will add random tests,
also CW going on notification break
yes, lately I receive around 50% of notifications :s
done.
resolved
This comment has been hidden.
This comment has been hidden.
This comment has been hidden.
Thanks for pointing that out :)