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

Bad error message when entering code with multiple holes and code with syntax error #21

Open
swaratchaudhuri opened this issue Jul 19, 2017 · 3 comments
Assignees

Comments

@swaratchaudhuri
Copy link
Collaborator

swaratchaudhuri commented Jul 19, 2017

(This is closely related to issue #16.)

Here is some code I entered in Askbayou:

import edu.rice.cs.caper.bayou.annotations.Evidence;

public class TestIO1 {
    void read(String file) {
        Evidence.apicalls("readLine");
    }     
    void foo(String file) {
        Evidence.apicalls("readLine");
    }  
}

This leads to to two bad things:

  1. The message "Unexpected error. Please retry." This is a bad error message. Bayou should return a message that clarifies that we only allow one hole in the code at one time, and Askbayou should deliver this message.

  2. The "synthesize" button goes away. Note that this doesn't happen on successful termination, and it shouldn't happen in this error case either.

It seems like the same sorts of problems arise when we have a syntax error in the draft:

import edu.rice.cs.caper.bayou.annotations.Evidence;
public class TestIO1 {
    // Read from a file
    void read(String file) {
        Evidence.apicalls("readLine")
    }   
}
@swaratchaudhuri swaratchaudhuri changed the title "Unexpected error. Please retry." message when entering code with multiple holes Bad error message when entering code with multiple holes Jul 19, 2017
@swaratchaudhuri swaratchaudhuri changed the title Bad error message when entering code with multiple holes Bad error message when entering code with multiple holes and code with syntax error Jul 19, 2017
@rmbarnett-rice rmbarnett-rice self-assigned this Jul 19, 2017
@rmbarnett-rice
Copy link
Collaborator

Commit 42fa463 addresses the search button not showing after an unexpected error. This was an artefact of the old UI where the error messages and search button were in the same place.

@rmbarnett-rice
Copy link
Collaborator

This is related to trishullab/bayou#41

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants
@rmbarnett-rice @swaratchaudhuri and others