Skip to content

Fix soft button object invalid states #1776

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.

Sign up for GitHub

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

Merged
merged 20 commits into from
Feb 3, 2022
Merged

Fix soft button object invalid states #1776

Changes from 1 commit
Commits
Show all changes
20 commits
Select commit Hold shift + click to select a range
2445b37
Resolve #1774
noah-livio Dec 7, 2021
72e24b5
Add tests
noah-livio Dec 7, 2021
c9da956
Update Javadoc with throws annotations
noah-livio Dec 7, 2021
0aebb4e
Removing throwing of IllegalStateException
noah-livio Dec 14, 2021
b00cea5
Remove unused import
noah-livio Dec 14, 2021
8413d94
Make changes discussed in #1776 review
noah-livio Dec 15, 2021
f95aec3
Fix a small typo in the unit test javadoc
noah-livio Dec 15, 2021
648bdc3
Cleanup code and modify tests
noah-livio Dec 17, 2021
9a82059
Move states assignment back to original line
noah-livio Dec 17, 2021
053ac1b
Add duplicate name check to setStates()
noah-livio Dec 17, 2021
9d47512
Add AssertionErrors as per iOS alignment
noah-livio Jan 7, 2022
386027f
Remove redundant conditions and commented code
noah-livio Jan 7, 2022
75d1e1e
Add AssertionError to SoftButtonState
noah-livio Jan 7, 2022
d15498f
Add test for initialStateName check
noah-livio Jan 10, 2022
34c0622
Revert addtion of AssertionErrors
noah-livio Jan 11, 2022
528769f
Update SoftButtonManagerTests test
noah-livio Jan 20, 2022
e183f35
Update ScreenManagerTests test
noah-livio Jan 20, 2022
666f99a
Fixes from PR feedback
noah-livio Jan 26, 2022
14b7569
Review and standup feedback fixes
noah-livio Jan 28, 2022
39efd8e
PR feedback change
noah-livio Feb 1, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Add AssertionError to SoftButtonState
  • Loading branch information
@noah-livio
noah-livio committed Jan 7, 2022
commit 75d1e1e7a1bafd2c07c586f6f653cb1447ff833d
3 changes: 3 additions & 0 deletions base/src/main/java/com/smartdevicelink/managers/screen/SoftButtonState.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,7 @@

import androidx.annotation.NonNull;

import com.livio.BuildConfig;
Copy link
Contributor

@RHenigan RHenigan Jan 24, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove unused import

import com.smartdevicelink.managers.file.filetypes.SdlArtwork;
import com.smartdevicelink.proxy.rpc.SoftButton;
import com.smartdevicelink.proxy.rpc.enums.SoftButtonType;
Expand Down Expand Up @@ -65,6 +66,8 @@ public SoftButtonState(@NonNull String name, String text, SdlArtwork artwork) {
if (text == null && artwork == null) {
DebugTool.logError(TAG, "Attempted to create an invalid soft button state: text and artwork are both null");
softButton = null;
if (BuildConfig.DEBUG)
throw new AssertionError("Attempted to create an invalid soft button state: text and artwork are both null");
return;
}
this.name = name;
Expand Down