-
Notifications
You must be signed in to change notification settings - Fork 3
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
Bug-5 record module #58
base: master
Are you sure you want to change the base?
Conversation
@thomas771020 The first issue is the travis is not passed. Please check that. |
import android.media.MediaRecorder; | ||
import android.util.Log; | ||
|
||
public class RecordUtils { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would suggest to rename this class as RecorderUtils
. That sounds more natural.
It looks good overall. Please update the patch and let's see this again. |
c29a284
to
79458c4
Compare
79458c4
to
96e6c85
Compare
Hi @john-hu I have updated this one. Please help me review it again. Thank you. |
} | ||
}); | ||
getInstrumentation().waitForIdleSync(); | ||
performClick(dialog.getButton(AlertDialog.BUTTON_POSITIVE)); | ||
assertEquals(20, mCardActivity.getCardPosition()); | ||
assertEquals(0, mCardActivity.getCardPosition()); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sine change card index was caused other test fail. So I change test process to avoid it.
Hi, @john-hu This Patch is for record word feature. Please help me review it. Thank you~