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

[Cleanup] Moving a bunch of classes into android package (pt3). #204

Merged
Show file tree
Hide file tree
Changes from all commits
Commits
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
9 changes: 9 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,15 @@
- Move `FragmentKey` into `android` package.
- Move BaseFormulaFragment into `android` package.
- Move `DisposableScope` into `android` package.
- Move `FormulaFragment` into `android` package.
- Move `BackCallback` into `android` package.
- Move `FragmentLifecycleEvent` into `android.events` package.
- Move `Integration` into `android` package.
- Move `FragmentLifecycleCallback` into `android` package.
- Move `FragmentComponent` into `android` package.
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I accidentally merged #205 into this PR as well. Sorry @Jawnnypoo

Copy link
Member

Choose a reason for hiding this comment

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

Oops, no worries

- Move `FragmentContract` into `android` package.
- Move `FragmentFlowState` into `android` package.
- Move `FragmentFlowStore` into `android` package.

## [0.6.1] - November 18, 2020
- Bugfix: Fix runtime ignoring `Formula.key` for the root formula.
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,8 @@ package com.instacart.formula.test

import android.view.View
import com.instacart.formula.R
import com.instacart.formula.fragment.FragmentComponent
import com.instacart.formula.fragment.FragmentContract
import com.instacart.formula.android.FragmentComponent
import com.instacart.formula.android.FragmentContract
import kotlinx.android.parcel.Parcelize

@Parcelize
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,8 @@ package com.instacart.formula.test

import android.view.View
import com.instacart.formula.R
import com.instacart.formula.fragment.FragmentComponent
import com.instacart.formula.fragment.FragmentContract
import com.instacart.formula.android.FragmentComponent
import com.instacart.formula.android.FragmentContract
import kotlinx.android.parcel.Parcelize

@Parcelize
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,8 +4,8 @@ import android.os.Bundle
import androidx.annotation.VisibleForTesting
import com.instacart.formula.FormulaAndroid
import com.instacart.formula.R
import com.instacart.formula.fragment.FormulaFragment
import com.instacart.formula.fragment.FragmentContract
import com.instacart.formula.android.FormulaFragment
import com.instacart.formula.android.FragmentContract
import com.instacart.formula.android.FragmentKey
import com.instacart.formula.android.FormulaAppCompatActivity

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,9 +3,9 @@ package com.instacart.formula.test
import android.view.View
import com.instacart.formula.RenderView
import com.instacart.formula.Renderer
import com.instacart.formula.fragment.FragmentComponent
import com.instacart.formula.fragment.FragmentContract
import com.instacart.formula.fragment.FragmentLifecycleCallback
import com.instacart.formula.android.FragmentComponent
import com.instacart.formula.android.FragmentContract
import com.instacart.formula.android.FragmentLifecycleCallback

object TestFragmentComponent {

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,9 +3,9 @@ package com.instacart.formula.test
import android.os.Bundle
import android.view.View
import com.instacart.formula.R
import com.instacart.formula.fragment.FragmentComponent
import com.instacart.formula.fragment.FragmentContract
import com.instacart.formula.fragment.FragmentLifecycleCallback
import com.instacart.formula.android.FragmentComponent
import com.instacart.formula.android.FragmentContract
import com.instacart.formula.android.FragmentLifecycleCallback
import kotlinx.android.parcel.IgnoredOnParcel
import kotlinx.android.parcel.Parcelize

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -6,10 +6,10 @@ import androidx.test.core.app.ActivityScenario
import androidx.test.ext.junit.rules.ActivityScenarioRule
import androidx.test.ext.junit.runners.AndroidJUnit4
import com.google.common.truth.Truth.assertThat
import com.instacart.formula.fragment.FragmentContract
import com.instacart.formula.fragment.FragmentFlowState
import com.instacart.formula.android.FragmentContract
import com.instacart.formula.android.FragmentFlowState
import com.instacart.formula.android.FragmentKey
import com.instacart.formula.integration.BackCallback
import com.instacart.formula.android.BackCallback
import com.instacart.formula.test.TestContract
import com.instacart.formula.test.TestContractWithId
import com.instacart.formula.test.TestFragmentActivity
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,8 +4,8 @@ import androidx.test.core.app.ActivityScenario
import androidx.test.ext.junit.rules.ActivityScenarioRule
import androidx.test.ext.junit.runners.AndroidJUnit4
import com.google.common.truth.Truth.assertThat
import com.instacart.formula.fragment.FormulaFragment
import com.instacart.formula.fragment.FragmentContract
import com.instacart.formula.android.FormulaFragment
import com.instacart.formula.android.FragmentContract
import com.instacart.formula.android.ActivityStoreContext
import com.instacart.formula.test.TestContract
import com.instacart.formula.test.TestContractWithId
Expand Down
Original file line number Diff line number Diff line change
@@ -1,9 +1,7 @@
package com.instacart.formula.android

import androidx.fragment.app.FragmentActivity
import com.instacart.formula.fragment.FragmentFlowState
import com.instacart.formula.fragment.FragmentFlowStore
import com.instacart.formula.fragment.FragmentLifecycleEvent
import com.instacart.formula.android.events.FragmentLifecycleEvent
import io.reactivex.rxjava3.disposables.Disposable

/**
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,9 +3,7 @@ package com.instacart.formula.android
import androidx.fragment.app.FragmentActivity
import androidx.lifecycle.Lifecycle
import com.instacart.formula.android.events.ActivityResult
import com.instacart.formula.fragment.FragmentFlowState
import com.instacart.formula.fragment.FragmentFlowStore
import com.instacart.formula.fragment.FragmentLifecycleEvent
import com.instacart.formula.android.events.FragmentLifecycleEvent
import io.reactivex.rxjava3.core.Observable
import io.reactivex.rxjava3.disposables.Disposable

Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
package com.instacart.formula.integration
package com.instacart.formula.android

/**
* Used to indicate that a screen render model
Expand Down
Original file line number Diff line number Diff line change
@@ -1,8 +1,6 @@
package com.instacart.formula.android

import io.reactivex.rxjava3.core.Observable
import com.instacart.formula.fragment.FormulaFragment
import com.instacart.formula.fragment.FragmentFlowStore

/**
* Feature is based on uni-directional state management where a single state model drives
Expand Down
Original file line number Diff line number Diff line change
@@ -1,8 +1,5 @@
package com.instacart.formula.android

import com.instacart.formula.fragment.FormulaFragment
import com.instacart.formula.fragment.FragmentFlowStore

/**
* Feature factory is responsible for creating a [Feature] for a specific [fragment key][Key].
*
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,6 @@ package com.instacart.formula.android

import android.view.View
import com.instacart.formula.Cancelable
import com.instacart.formula.fragment.FragmentLifecycleCallback
import com.instacart.formula.fragment.FormulaFragment
import io.reactivex.rxjava3.core.Observable

/**
Expand Down
Original file line number Diff line number Diff line change
@@ -1,16 +1,11 @@
package com.instacart.formula.fragment
package com.instacart.formula.android

import android.os.Bundle
import android.view.LayoutInflater
import android.view.View
import android.view.ViewGroup
import androidx.fragment.app.Fragment
import com.instacart.formula.Cancelable
import com.instacart.formula.android.BaseFormulaFragment
import com.instacart.formula.android.FeatureView
import com.instacart.formula.android.FragmentEnvironment
import com.instacart.formula.android.FragmentKey
import com.instacart.formula.android.ViewFactory
import com.jakewharton.rxrelay3.BehaviorRelay

class FormulaFragment : Fragment(), BaseFormulaFragment<Any> {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,9 +4,7 @@ import com.instacart.formula.android.internal.Binding
import com.instacart.formula.android.internal.Bindings
import com.instacart.formula.android.internal.FunctionUtils
import com.instacart.formula.android.views.FragmentContractViewFactory
import com.instacart.formula.fragment.FragmentContract
import com.instacart.formula.android.internal.FeatureBinding
import com.instacart.formula.integration.Integration
import io.reactivex.rxjava3.core.Observable
import java.lang.IllegalStateException
import kotlin.reflect.KClass
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
package com.instacart.formula.fragment
package com.instacart.formula.android

import com.instacart.formula.Renderer
import com.instacart.formula.RenderView
Expand Down
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
package com.instacart.formula.fragment
package com.instacart.formula.android

import android.view.View
import com.instacart.formula.android.FragmentKey

/**
* Responsible for providing a [FragmentComponent]
Expand Down
Original file line number Diff line number Diff line change
@@ -1,9 +1,4 @@
package com.instacart.formula.fragment

import com.instacart.formula.android.FragmentId
import com.instacart.formula.android.FeatureEvent
import com.instacart.formula.android.FragmentKey
import com.instacart.formula.android.FragmentState
package com.instacart.formula.android

/**
* Represents currently [activeKeys] and their [states].
Expand Down
Original file line number Diff line number Diff line change
@@ -1,16 +1,11 @@
package com.instacart.formula.fragment
package com.instacart.formula.android

import com.instacart.formula.Evaluation
import com.instacart.formula.Formula
import com.instacart.formula.FormulaContext
import com.instacart.formula.android.FragmentId
import com.instacart.formula.rxjava3.RxStream
import com.instacart.formula.android.internal.Binding
import com.instacart.formula.android.FeatureEvent
import com.instacart.formula.android.DisposableScope
import com.instacart.formula.android.FragmentBindingBuilder
import com.instacart.formula.android.FragmentEnvironment
import com.instacart.formula.android.FragmentState
import com.instacart.formula.android.events.FragmentLifecycleEvent
import com.instacart.formula.rxjava3.toObservable
import com.jakewharton.rxrelay3.PublishRelay
import io.reactivex.rxjava3.core.Observable
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
package com.instacart.formula.fragment
package com.instacart.formula.android

import android.os.Bundle
import android.view.View
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
package com.instacart.formula.integration
package com.instacart.formula.android

import io.reactivex.rxjava3.core.Observable

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@ package com.instacart.formula.android
import android.view.LayoutInflater
import android.view.ViewGroup
import androidx.annotation.LayoutRes
import com.instacart.formula.fragment.FormulaFragment
import com.instacart.formula.android.views.DelegateLayoutViewFactory

/**
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,8 +4,6 @@ import android.view.View
import com.instacart.formula.RenderView
import com.instacart.formula.Renderer
import com.instacart.formula.android.views.FeatureViewBindFunction
import com.instacart.formula.fragment.FormulaFragment
import com.instacart.formula.fragment.FragmentLifecycleCallback

/**
* View instance contains an initialized Android [view] and provides factory
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
package com.instacart.formula.fragment
package com.instacart.formula.android.events

import com.instacart.formula.android.FragmentId

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ package com.instacart.formula.android.internal
import androidx.fragment.app.FragmentActivity
import androidx.lifecycle.Lifecycle
import com.instacart.formula.android.events.ActivityResult
import com.instacart.formula.fragment.FragmentFlowState
import com.instacart.formula.android.FragmentFlowState
import com.instacart.formula.android.FragmentKey
import com.instacart.formula.android.FragmentId
import com.instacart.formula.android.ActivityStoreContext
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,11 +10,11 @@ import androidx.lifecycle.Lifecycle
import com.instacart.formula.Renderer
import com.instacart.formula.RenderView
import com.instacart.formula.android.BaseFormulaFragment
import com.instacart.formula.fragment.FormulaFragment
import com.instacart.formula.android.FormulaFragment
import com.instacart.formula.android.FragmentEnvironment
import com.instacart.formula.fragment.FragmentFlowState
import com.instacart.formula.fragment.FragmentLifecycleEvent
import com.instacart.formula.integration.BackCallback
import com.instacart.formula.android.FragmentFlowState
import com.instacart.formula.android.events.FragmentLifecycleEvent
import com.instacart.formula.android.BackCallback
import com.instacart.formula.android.FeatureEvent
import com.instacart.formula.android.FragmentId
import java.util.LinkedList
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,8 @@ import androidx.fragment.app.FragmentManager
import com.instacart.formula.android.FragmentId
import com.instacart.formula.android.FragmentKey
import com.instacart.formula.android.BaseFormulaFragment
import com.instacart.formula.fragment.FormulaFragment
import com.instacart.formula.fragment.FragmentLifecycleEvent
import com.instacart.formula.android.FormulaFragment
import com.instacart.formula.android.events.FragmentLifecycleEvent

/**
* Provides utility method [lifecycleEvents] to track what fragments are added and removed from the backstack.
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ import com.instacart.formula.android.FeatureView
import com.instacart.formula.android.LayoutViewFactory
import com.instacart.formula.android.ViewFactory
import com.instacart.formula.android.ViewInstance
import com.instacart.formula.fragment.FragmentContract
import com.instacart.formula.android.FragmentContract

/**
* An implementation of [ViewFactory] which delegates implementation to [FragmentContract].
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@ package com.instacart.formula.android
import androidx.fragment.app.FragmentActivity
import androidx.lifecycle.Lifecycle
import com.instacart.formula.android.internal.ActivityStoreContextImpl
import com.instacart.formula.fragment.FragmentContract
import com.jakewharton.rxrelay3.PublishRelay
import com.nhaarman.mockito_kotlin.mock
import com.nhaarman.mockito_kotlin.verify
Expand Down
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
package com.instacart.formula.android

import com.google.common.truth.Truth
import com.instacart.formula.fragment.FragmentFlowStore
import com.instacart.formula.android.fakes.TestAccountFragmentContract
import com.instacart.formula.android.fakes.TestLoginFragmentContract
import com.instacart.formula.android.fakes.TestSignUpFragmentContract
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,9 +5,7 @@ import com.instacart.formula.android.fakes.DetailKey
import com.instacart.formula.android.fakes.FakeAuthFlowFactory
import com.instacart.formula.android.fakes.FakeComponent
import com.instacart.formula.android.fakes.MainKey
import com.instacart.formula.fragment.FragmentContract
import com.instacart.formula.fragment.FragmentFlowStore
import com.instacart.formula.fragment.FragmentLifecycleEvent
import com.instacart.formula.android.events.FragmentLifecycleEvent
import com.instacart.formula.android.fakes.TestAccountFragmentContract
import com.instacart.formula.android.fakes.TestLoginFragmentContract
import com.instacart.formula.android.fakes.TestSignUpFragmentContract
Expand Down
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
package com.instacart.formula.android.fakes

import android.view.View
import com.instacart.formula.fragment.FragmentComponent
import com.instacart.formula.fragment.FragmentContract
import com.instacart.formula.android.FragmentComponent
import com.instacart.formula.android.FragmentContract
import kotlinx.android.parcel.Parcelize

@Parcelize
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ package com.instacart.formula.android.fakes

import com.instacart.formula.android.Flow
import com.instacart.formula.android.FlowFactory
import com.instacart.formula.fragment.FragmentContract
import com.instacart.formula.android.FragmentContract
import com.instacart.formula.android.DisposableScope
import io.reactivex.rxjava3.core.Observable

Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
package com.instacart.formula.android.fakes

import com.instacart.formula.fragment.FragmentContract
import com.instacart.formula.android.FragmentContract
import com.instacart.formula.android.FragmentKey
import com.instacart.formula.android.DisposableScope
import com.jakewharton.rxrelay3.PublishRelay
Expand Down
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
package com.instacart.formula.android.fakes

import android.view.View
import com.instacart.formula.fragment.FragmentComponent
import com.instacart.formula.fragment.FragmentContract
import com.instacart.formula.android.FragmentComponent
import com.instacart.formula.android.FragmentContract
import kotlinx.android.parcel.Parcelize

@Parcelize
Expand Down
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
package com.instacart.formula.android.fakes

import android.view.View
import com.instacart.formula.fragment.FragmentComponent
import com.instacart.formula.fragment.FragmentContract
import com.instacart.formula.android.FragmentComponent
import com.instacart.formula.android.FragmentContract
import kotlinx.android.parcel.Parcelize

@Parcelize
Expand Down
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
package com.instacart.formula.android.fakes

import android.view.View
import com.instacart.formula.fragment.FragmentComponent
import com.instacart.formula.fragment.FragmentContract
import com.instacart.formula.android.FragmentComponent
import com.instacart.formula.android.FragmentContract
import kotlinx.android.parcel.Parcelize

@Parcelize
Expand Down
Loading