The default parameter here is called "word", which is confusing because the function expects a sentence.

Repro steps:

1. Open app.html on a browser (chrome used in above screenshot)

How to reproduce:

1. Open app.html with Chrome in Ubuntu 14.04;
2. In question 1, select the text box where to enter the code;
3. Without typing anything, press ctrl+z (the shortcut to rescind some edit), one could get what shows in the screenshot.

Reproduction Steps:

1. Open the app.html file in chrome
2. Go to Sample Question 1 (Any sample question would do)
3. Add a long line of code that extends beyond the visible area
4. Scroll to the extreme right using the horizontal scroll bar

Expected Output: The line numbers should not overlap with the code

Actual Output: The code contents overlap with the line numbers on the left

The problem statement is not so clear. We don't know what to expect as the result of the decoder. The problem said something about one-to-one mapping between the encoder and decoder, 23xa means 2aaa and not 23 of 'a'. However, it is not just clear in the problem statement.

Behaviour

The text areas with the instruction and the question can be resized by the user.

Expected behaviour

Text areas resize or become scrollable based on the amount of text in them (with a fixed minimum size)

Repro steps

1. Open app.html
2. Click and drag from the marker on the bottom right corner of the instuction/question box

Expected behavior: No errors seen in Dev Tools console.

Observed behavior: "TypeError: Cannot read property 'forEach' of undefined" error written to Dev Tools console after completing the first question.

Steps to reproduce:

1. Chrome Open Developer Tools console
2. Complete first question
3. Observe error

Since the test cases, which are quite few and rather simple, are shown on screen with their correct answers once failed, challengers can list out all test cases as predicates in their codes and 'cheat'.

Repro steps:

1. Open app.html
2. Navigate to the i18n problem.
3. Change "abbreviate" to "isBalanced"
4. Run code.
5. You should see this error: ["Your code threw an error: AttributeError: 'StudentCode' object has no attribute 'abbreviate' on line 42"]

It might be worth catching that error and saying something closer to "You seem to be missing an 'abbreviate' method. Check the starter code and make sure that it hasn't been deleted or modified accidentally."

I wanted to print something to the screen with print(), but that did not work. I suggest adding stdout to the page, since that could be useful sometime.

Reproduction Steps:

1. Open "app.html"

2. Navigate to a question

3. Add a library import like import string or import re

4. Use the import in the function like word[0] in string.whitespace or re.compile(r'\w')

5. Click Run

Current message is:
NameError: name 'string' is not defined on line 3

Which could be:
Library imports are not permitted for this problem

Standard library imports like re and string may not be allowed as they make the problem solution too simple and unchallenging. However, it might be a god idea to say that explicitly.

[EDIT Mar 4, 11:24]
Suggested Solutions:
Option 1: Ban the use all libraries for all questions
This is a blanket solution. For each solution to any problem, check for an import statement. If there is one, don't compile the problem. Instead, show a message saying imports are not permitted.
This is how the system is currently designed and would only require adding a check and a message. This can be a good minimal solution.

Option 2: Ban libraries specific to questions
We could have two sets of libraries - banned in every question e.g. pandas and banned for this specific question e.g. re for string manipulation questions. This is a more elegant and sustainable solution and requires a little more work than the first option

Repro Steps:

1. Open "app.html".

2. Navigate to question 1.

3. Change the returned string from "" to a random string(e.g., "abcd") which will produce wrong answer.

4. Click "Run".

5. The console now shows the input("moo cow bark dog") and the output("abcd"), and the expected output("oom woc krab god") if scrolling down the window.
   

6. Change the returned string from "abcd" to another random string(suppose the user is fixing the bug in the code).

7. Run the code again.

8. The console shows the expected output, but the actual output is not seen until you scroll up the window.
   

Expected behavior: The console always displays the input and actual output, whenever the user runs the code. It's easier for the user to see what his/her code produced and debug the code.

Executing code after changing the function name displays error and breaks the output console.

Reproduction Steps:

1. Open the app.html file in chrome
2. Go to Sample Question 1 (Any sample question would do)
3. Change the function name to any name of your choice
4. Press 'Run'

Working on creating this question in branch: bomberman_question

Pressing undo in the editor from the default text makes strange text appear in the editor.

Repro steps:

1. Open app.html
2. Click on editor and press undo (command-Z used here on a mac)

Create Most Common Character Question with @spwahaha

The Judge does not test my code for tasks that I've passed before.

For example, on Question 2, my code went through Task One smoothly but had bugs on Task Two. Then I made some changes so as to pass the tests for Task Two. Even through now my code failed on the tests of Task One, the Online Judge would not find out, but instead moved me forward to the third task. Such design seemed quite inappropriate to me, because the next round task, based on the previous one, encourages challengers to generalize their codes, rather than allowing them to pass the tasks individually.

In the description of question 4:

"abcccccd" should be encoded as "ab5xcd" but not "ab5xc"

Per discussion in #57, let's add the new CodeSubmission object to the Snapshot (that's used as part of a transcript) so that the transcript has all the information needed to restore a user session.

Behaviour

On a larger window, everything is centered with the same height on the left and right sides:

On smaller windows, the elements do not have a consistent width, nor are they centered on the page. Also, the extra height on these is not required when the elements are stacked vertically.

Repro steps

1. Open app.html
2. Decrease browser window size till the question box goes below the code editing box

You told me to write this.

Hi,

The current version of TIE does not do auto-indentation when there is a new line after 'def', 'if' or 'where' clauses. It just aligns the new line with the previous one. It may be more user-friendly and save some time if this feature is included, since it is not convenient for users to count spaces themselves.

I captured two screenshots to compare between TIE and Leetcode.

Retro steps:
In step3 of Question 4, Run-Length Encoding:
We write a one-line program to return the input directly, then click 'run'.

Output:
The output of our function will be 'bbbbb' when input is '5xb'.

Expected behavior:
The output of our function will be '5xb' when input is '5xb'.

Possible causes:
'bbbbb' is the result after using the function 'AuxiliaryCode.decodeEncodedString' defined in 'outputFunctionName' attribute in JS script of questions on our output '5xb'. It may confuse some users cause the output in the console is not the actual output of their function.

Suggestion:
Maybe we can modify the text when outputFunctionName != null. For this question, a possible output could be 'Output after decoding:', 'Expected Output after decoding'. Or we can modify the question: give them the decoding function/strategy to ask them write correct encoding function to make sure the string remain the same after encoding and decoding. Or in the last step we can ask them to implement the decoding function by themselves.

Repro steps from scrach:

Start any question
Complete a task within that question
Prompt will have inconsistent message with button -- button should change, I think.

If you try the following in the reverse word question:

def reverseWords(word):
  if word == "moo cow bark dog":
    return "oom woc krab god"

Parsing fails and an error is written to console, but no feedback (or error) is given to the user.

Expected behavior: Some indication to user what went wrong.

Observed behavior: Nothing (unless you open a console and see the obscure error message written there)

Any output function defined in the auxiliary code and then specified in the data schema (such as rle.js's decodeEncodedString) is not currently used.

Repro steps:

1. Go to the run-length encoding problem, once I've pushed my fixes.
2. Paste in the solution.
3. Run until you get to the "decode" step.
4. Your output should return 1x5xb for one correctness test, rather than 5xb, signifying that AuxiliaryCode.decodeEncodedString was not run on the output.

For are_all_unique, it takes in an array as an argument, but that's not currently supported, so jsonVariableToPython crashes.

Launch app.html
Navigate to i18n question
Complete tasks up to first instance of filling in are_all_unique
App should crash.

If we make the function always return false, it says "Try running your code on '))((' on paper. Did you expect that result?". This suggestion is useless, because the input '))((' should get false.

I finished the question 2 (compiled successfully and got tests passed) and moved forward to question 3. When I went back to question 2 (click "2" in the process bar) to review the code, the code has been reset and my code was not saved.

On the GUI it said "You'll see the first question to your right". However, the question appears in the bottom.

We should probably add an end-to-end test that loads the page and clicks some of the buttons / submits some code, in order to prevent breakages like the one that was fixed in #80. (Note that even an end-to-end test which just loads the app.html page in a browser and checks that there are no console errors would have helped here.)

Protractor is likely to be a good library for this, and is worth investigating.

Sometimes users want to create custom test cases to figure out the bugs in their codes, but there is no place to do it in the webpage right now. So it should be helpful to have it and maybe it's better to embed a default right algorithm for each problem to show the correct result for the custom test case.

The description/examples of this question could be more detailed, as dog->d1g is a valid case according to the description but failed here:

Maybe like below:
An abbreviation of a word follows the form <first character><length of the middle of the string><last character>. Below are some examples of word abbreviations:
a) it --> it (no abbreviation)
b) d|o|g --> dog (no abbreviation)
c) i|nternationalizatio|n --> i18n

When I tried to import some packages, I got an ImportError. Here is an example,

The output screen at the left top corner has a black box to indicate the Run results, which only scales based on the screen size instead of the text length. Thus, if the text within this black box is longer than screen size, the rest text will simply extend out of the black box, which is not very elegant.

Repro instructions:

1. Run app.html
2. Navigate to Question-1
3. Change code to return a very long string, such as

4. Click on Run button
5. Actual result will come out in this black box

We can find out the output text exceeds the actual size of the black box.

(This comes from a suggestion in #88.)

It would be nice to have some sort of way for students to "ask questions". This could be as simple as having a question mark button in the bottom right and allowing them to select, multiple-choice style, clarifying questions from it (including "can I have a hint, please?"). That way, although they might not be able to practice coming up with questions on their own, at least they get a feel for what sort of thing to ask.

Currently, if user navigate to another problem, they will lose all the work on current problem even without a warning message.

1. Do some work on Q2
   

2. Click question 3
   

3. Then you lose all your work on Q2
   

This can be very frustrating for users if they just navigate to another problem by accident, or they are stuck on the current one and they wish to work on another problem and come back later. We might keep user's work in session or at least add an alert message warning that they are about to lose all the current work.

In the last phase of this question, the user are required to encode the word in a comprehensible manner, in which "5aaaa" can be decoded properly.
However, the most straight forward way is to encode "5" as "1x5", which conflict with the previous phase's requirement(encoding should be as short as possible).

Expected behavior: When question or feedback looks at how whitespace is handled, whitespace is rendered properly in the feedback window.

Observed behavior: Whitespace appears as a single space character, regardless of whether it is a tab or consists of multiple whitespace characters.

Steps to reproduce:

1. Try reverse string exercise
2. Fail the whitespace prompt
3. Look at feedback

Repro instructions: Go to the reverseWords problem, and type the code seen in the screenshot below.

Observed behaviour: Runtime errors have the wrong line numbers. For example, in the screenshot below, the error is shown as being on line 15, whereas the actual error is on line 3.

Expected behaviour: The error message should show the correct line number.

Students should not be able to access AuxiliaryCode or System classes.

Repro steps (at least, the easiest ones):

Navigate to Problem 2
Paste in the following code:
def isBalanced(s): return AuxiliaryCode.countNumberOfParentheses(s)
Run code. You should get an error message (rather than a SyntaxError or something) since it called one of the AuxiliaryCode methods. We should block students from calling any of those classes, potentially via throwing a SyntaxError or a ForbiddenNamespaceError of our own making. Something to tell them yo you can't run our code.

1. Cannot add a character in middle.
   When users want to add 'c' in middle of 'abde', users will expect it becomes 'abcde'. But the actual result turns to be 'abce', where the new character 'c' covers the old character 'd' instead of adding itself between 'b' and 'd' in 'abde'.

2. Cannot scale as two other output screens.

Repro instructions: Go to the reverseWords problem, and type the code seen in the screenshot below.

In the screenshot below, the feedback message is not interpolated, so it is meaningless. We should either support interpolation or choose a different feedback message.

For some reason the editor is conflating tabs and spaces and it's causing all sorts of errors. I usually run into this on the i18n question when adding the length check to abbreviate.

Open app.html
Copy/paste solution.
Add lines to solution (hitting "Enter" after a complete line), and then tab inwards.
You see the correct indentation, but it's now mixed tabs with spaces.

In the code, we just return Booleans such as False, False or False, False or False or False...and so on. However, the hints got different every time. Seems like the hints are just iterating in the back-ends regardless of the actual mistake of user's code.

And if we return anything but an expression that results in False, such as None, True...It displays the input, output and the expected output. I kind wish it always displays so that users know what input they are tested against.