chore: update visibility modifiers (#993)

* Add InternalOudsApi annotation to token files in theme-contract

* Token data classes in theme-orange are now internal

* Token data classes in theme-sosh are now internal

* Token data classes in theme-wireframe are now internal

* Move InputStream.contentAsString extension method in app module

* Most of test related classes are now internal

* Rename OudsLink arrow to chevron

* OudsColorMode is now internal

* Change visibility of several properties, methods and classes of core module to internal

* getDefaultIconPainter in OudsTag is now internal

* Review: Remove internal modifier on constructors of OudsColorScheme.Repository and OudsColorScheme.Modes nested classes

* Review: Use isSupported on OudsColoredBoxColor instead of OudsColorMode whenever possible

* Review: getDefaultIconPainter is not abstract anymore and provides a default implementation

Author: florentmaitre

app/src/main/java/com/orange/ouds/app/ui/components/ComponentIllustrations.kt

@@ -42,7 +42,7 @@ import com.orange.ouds.core.component.OudsColoredBox
 import com.orange.ouds.core.component.OudsFilterChip
 import com.orange.ouds.core.component.OudsHorizontalDivider
 import com.orange.ouds.core.component.OudsLink
-import com.orange.ouds.core.component.OudsLinkArrow
+import com.orange.ouds.core.component.OudsLinkChevron
 import com.orange.ouds.core.component.OudsNavigationBar
 import com.orange.ouds.core.component.OudsNavigationBarItem
 import com.orange.ouds.core.component.OudsNavigationBarItemIcon
@@ -131,7 +131,7 @@ fun DividerIllustration() = ComponentIllustration {
 fun LinkIllustration() = ComponentIllustration {
     OudsLink(
         label = stringResource(id = R.string.app_components_common_label_label),
-        arrow = OudsLinkArrow.Next,
+        chevron = OudsLinkChevron.Next,
         onClick = {}
     )
 }

app/src/main/java/com/orange/ouds/app/ui/components/coloredbackground/ColoredBackgroundDemoScreen.kt

@@ -41,7 +41,7 @@ import com.orange.ouds.core.component.OudsButton
 import com.orange.ouds.core.component.OudsColoredBox
 import com.orange.ouds.core.component.OudsColoredBoxColor
 import com.orange.ouds.core.component.OudsLink
-import com.orange.ouds.core.component.OudsLinkArrow
+import com.orange.ouds.core.component.OudsLinkChevron
 import com.orange.ouds.core.theme.OudsTheme
 
 @Composable
@@ -58,7 +58,7 @@ fun ColoredBackgroundDemoScreen() {
 @Composable
 private fun ColoredBackgroundDemoBottomSheetContent(state: ColoredBackgroundDemoState) {
     with(state) {
-        val colors = OudsColoredBoxColor.entries.filter { it.mode.isSupported }
+        val colors = OudsColoredBoxColor.entries.filter { it.isSupported }
         CustomizationDropdownMenu(
             applyTopPadding = false,
             label = stringResource(id = R.string.app_components_coloredBackground_color_label),
@@ -83,7 +83,7 @@ private fun ColoredBackgroundDemoBottomSheetContent(state: ColoredBackgroundDemo
 @Composable
 private fun ColoredBackgroundDemoContent(state: ColoredBackgroundDemoState) {
     with(state) {
-        if (!color.mode.isSupported) {
+        if (!color.isSupported) {
             Toast.makeText(
                 LocalContext.current,
                 stringResource(id = R.string.app_components_coloredBackground_unsupportedColor_text, color.name.toSentenceCase()),
@@ -112,7 +112,7 @@ private fun ColoredBackgroundDemoContent(state: ColoredBackgroundDemoState) {
                 )
                 OudsLink(
                     label = stringResource(id = R.string.app_components_link_label),
-                    arrow = OudsLinkArrow.Next,
+                    chevron = OudsLinkChevron.Next,
                     onClick = {},
                 )
             }
@@ -139,7 +139,7 @@ private fun Code.Builder.coloredBackgroundDemoCodeSnippet(state: ColoredBackgrou
                         }
                         functionCall("OudsLink") {
                             labelArgument(R.string.app_components_link_label)
-                            typedArgument("arrow", OudsLinkArrow.Next)
+                            typedArgument("chevron", OudsLinkChevron.Next)
                             onClickArgument {}
                         }
                     }

app/src/main/java/com/orange/ouds/app/ui/components/coloredbackground/ColoredBackgroundDemoState.kt

@@ -45,6 +45,6 @@ object ColoredBackgroundDemoStateDefaults {
     val Color: OudsColoredBoxColor
         @Composable
         get() = OudsColoredBoxColor.BrandPrimary
-            .takeIf { it.mode.isSupported }
-            .orElse { OudsColoredBoxColor.entries.first { it.mode.isSupported } }
+            .takeIf { it.isSupported }
+            .orElse { OudsColoredBoxColor.entries.first { it.isSupported } }
 }

app/src/main/java/com/orange/ouds/app/ui/components/link/LinkDemoScreen.kt

@@ -32,7 +32,7 @@ import com.orange.ouds.app.ui.utilities.composable.CustomizationSwitchItem
 import com.orange.ouds.app.ui.utilities.composable.CustomizationTextField
 import com.orange.ouds.app.ui.utilities.composable.DemoScreen
 import com.orange.ouds.core.component.OudsLink
-import com.orange.ouds.core.component.OudsLinkArrow
+import com.orange.ouds.core.component.OudsLinkChevron
 import com.orange.ouds.core.component.OudsLinkIcon
 import com.orange.ouds.core.component.OudsLinkSize
 import com.orange.ouds.theme.OudsVersion
@@ -109,19 +109,19 @@ private fun LinkDemoContent(state: LinkDemoState) {
                     size = size
                 )
             }
-            LinkDemoState.Layout.ArrowBack -> {
+            LinkDemoState.Layout.ChevronBack -> {
                 OudsLink(
                     label = label,
-                    arrow = OudsLinkArrow.Back,
+                    chevron = OudsLinkChevron.Back,
                     onClick = {},
                     enabled = enabled,
                     size = size
                 )
             }
-            LinkDemoState.Layout.ArrowNext -> {
+            LinkDemoState.Layout.ChevronNext -> {
                 OudsLink(
                     label = label,
-                    arrow = OudsLinkArrow.Next,
+                    chevron = OudsLinkChevron.Next,
                     onClick = {},
                     enabled = enabled,
                     size = size
@@ -143,8 +143,8 @@ private fun Code.Builder.linkDemoCodeSnippet(state: LinkDemoState, themeDrawable
                             painterArgument(themeDrawableResources.tipsAndTricks)
                         }
                     }
-                    LinkDemoState.Layout.ArrowBack -> typedArgument("arrow", OudsLinkArrow.Back)
-                    LinkDemoState.Layout.ArrowNext -> typedArgument("arrow", OudsLinkArrow.Next)
+                    LinkDemoState.Layout.ChevronBack -> typedArgument("chevron", OudsLinkChevron.Back)
+                    LinkDemoState.Layout.ChevronNext -> typedArgument("chevron", OudsLinkChevron.Next)
                 }
                 onClickArgument()
                 enabledArgument(enabled)

app/src/main/java/com/orange/ouds/app/ui/components/link/LinkDemoState.kt

@@ -86,7 +86,7 @@ class LinkDemoState(
     enum class Layout(@StringRes val labelRes: Int) {
         TextOnly(R.string.app_components_common_textOnlyLayout_label),
         TextAndIcon(R.string.app_components_common_textAndIconLayout_label),
-        ArrowBack(R.string.app_components_link_backLayout_label),
-        ArrowNext(R.string.app_components_link_nextLayout_label)
+        ChevronBack(R.string.app_components_link_backLayout_label),
+        ChevronNext(R.string.app_components_link_nextLayout_label)
     }
 }

app/src/main/java/com/orange/ouds/app/ui/tokens/TokenCategoryDetailScreen.kt

@@ -268,9 +268,9 @@ private fun CodeColumn(codeExample: String, modifier: Modifier = Modifier) {
     var isExpanded by rememberSaveable { mutableStateOf(false) }
     val linkStateDescription = stringResource(if (isExpanded) R.string.app_common_expanded_a11y else R.string.app_common_collapsed_a11y)
     val linkContentDescription = stringResource(R.string.app_tokens_common_viewCodeExample_label)
-    val transition = updateTransition(targetState = isExpanded, label = "LinkArrowTransition")
-    val linkArrowRotation by transition.animateFloat(
-        label = "LinkArrowRotation",
+    val transition = updateTransition(targetState = isExpanded, label = "LinkChevronTransition")
+    val linkChevronRotation by transition.animateFloat(
+        label = "LinkChevronRotation",
         transitionSpec = {
             tween(durationMillis = 300, easing = FastOutSlowInEasing)
         }
@@ -300,7 +300,7 @@ private fun CodeColumn(codeExample: String, modifier: Modifier = Modifier) {
                     color = OudsTheme.colorScheme.content.default
                 )
                 Icon(
-                    modifier = Modifier.rotate(linkArrowRotation),
+                    modifier = Modifier.rotate(linkChevronRotation),
                     painter = painterResource(LocalThemeDrawableResources.current.formChevronDown),
                     tint = OudsTheme.colorScheme.content.brandPrimary,
                     contentDescription = null

app/src/main/java/com/orange/ouds/app/ui/utilities/WebViewExt.kt

@@ -15,7 +15,8 @@ package com.orange.ouds.app.ui.utilities
 import android.webkit.WebView
 import androidx.annotation.RawRes
 import com.orange.ouds.app.R
-import com.orange.ouds.foundation.extensions.contentAsString
+import java.io.InputStream
+import java.util.Scanner
 
 internal fun WebView.injectLightDarkModeCss(darkMode: Boolean) {
     injectCss(R.raw.base_style)
@@ -29,4 +30,9 @@ private fun WebView.injectCss(@RawRes cssResource: Int) {
     val javascript = String.format(injectCssFunction, css.trim { it <= ' ' })
     val code = "javascript:(function() { $javascript })()"
     loadUrl(code)
-}
+}
+
+private fun InputStream.contentAsString(): String? {
+    val scanner = Scanner(this).useDelimiter("\\A")
+    return if (scanner.hasNext()) scanner.next() else null
+}

core-test/src/main/java/com/orange/ouds/core/test/OudsBadgeTest.kt

@@ -18,7 +18,7 @@ import org.junit.runner.RunWith
 import org.junit.runners.Parameterized
 
 @RunWith(Enclosed::class)
-class OudsBadgeTest {
+internal class OudsBadgeTest {
 
     @RunWith(Parameterized::class)
     class Default(parameter: Any) : OudsComponentSnapshotTest(

core-test/src/main/java/com/orange/ouds/core/test/OudsButtonTest.kt

@@ -18,7 +18,7 @@ import org.junit.runner.RunWith
 import org.junit.runners.Parameterized
 
 @RunWith(Enclosed::class)
-class OudsButtonTest {
+internal class OudsButtonTest {
 
     @RunWith(Parameterized::class)
     class Default(parameter: Any) : OudsComponentSnapshotTest(

core-test/src/main/java/com/orange/ouds/core/test/OudsCheckboxItemTest.kt

@@ -18,7 +18,7 @@ import org.junit.runner.RunWith
 import org.junit.runners.Parameterized
 
 @RunWith(Enclosed::class)
-class OudsCheckboxItemTest {
+internal class OudsCheckboxItemTest {
 
     @RunWith(Parameterized::class)
     class Default(parameter: Any) : OudsComponentSnapshotTest(