Skip to content
This repository has been archived by the owner on Jan 7, 2022. It is now read-only.

selectros for unregistered communities and change logic in launch sag… #90

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

Dinek007
Copy link
Contributor

…a init

Copy link

@holmesworcester holmesworcester left a comment

Choose a reason for hiding this comment

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

I didn't totally get what's happening here. Why do we have to check multiple communities?

import { identitySelectors } from './identity.selectors';
import { Identity, identityReducer, IdentityState } from './identity.slice';

describe('communitiesSelectors', () => {

Choose a reason for hiding this comment

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

Is there a more readable description that would be appropriate here? What are we checking?


describe('communitiesSelectors', () => {
let store: Store;
const communityAlpha: Community = {

Choose a reason for hiding this comment

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

Are these different users or different communities? Why are we testing with different communities?

port: 0,
};

const identityAlpha: Identity = {

Choose a reason for hiding this comment

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

Are these users? If so, should we just call them users?

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

Successfully merging this pull request may close these issues.

2 participants