Skip to content
This repository has been archived by the owner on Mar 13, 2024. It is now read-only.

Eliminate yarn test spam #1024

Merged
merged 13 commits into from
Apr 3, 2018
Merged

Eliminate yarn test spam #1024

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
3604978
remove "Network status set to (true|false)" message
lieut-data Mar 27, 2018
d548f3d
work around UnhandledPromiseRejectionWarning errors from localforage
lieut-data Mar 28, 2018
c99562f
fix missing props in tests
lieut-data Mar 28, 2018
73b5fcb
skip passing showEmailAddress from ProfilePopver to the Popover boots…
lieut-data Mar 28, 2018
b331b43
eliminate unneeded warning in getPrefix
lieut-data Mar 28, 2018
9a49fde
fix spurious state in rhs reducer tests
lieut-data Mar 28, 2018
dc6d2a4
leverage jest-canvas-mock to avoid warnings from the doughnut chart
lieut-data Mar 28, 2018
b7c99d9
fail the build on console messages
lieut-data Mar 28, 2018
ab47e5b
Revert "skip passing showEmailAddress from ProfilePopver to the Popov…
lieut-data Apr 3, 2018
688c6b8
just remove showEmailAddress in the profile popover unit tests
lieut-data Apr 3, 2018
7642e80
define STORE_REHYDRATION_FAILED locally vs. in mattermost-redux, for now
lieut-data Apr 3, 2018
e480ec1
Merge remote-tracking branch 'origin/master' into eliminate-yarn-test…
lieut-data Apr 3, 2018
b209749
fix recently broken unit tests
lieut-data Apr 3, 2018
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
just remove showEmailAddress in the profile popover unit tests
  • Loading branch information
@lieut-data
lieut-data committed Apr 3, 2018
commit 688c6b856e5a93a40520aa443795139b4f5a7b78
2 changes: 0 additions & 2 deletions components/profile_popover/index.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,12 +8,10 @@ import ProfilePopover from './profile_popover.jsx';
function mapStateToProps(state, ownProps) {
const config = state.entities.general.config;

const showEmailAddress = config.ShowEmailAddress === 'true';
const enableWebrtc = config.EnableWebrtc === 'true';

return {
...ownProps,
showEmailAddress,
enableWebrtc,
};
}
Expand Down
3 changes: 0 additions & 3 deletions tests/plugins/__snapshots__/pluggable.test.jsx.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,6 @@ exports[`plugins/Pluggable should match snapshot with no overridden component 1`
enableWebrtc={true}
hasMention={false}
isRHS={false}
showEmailAddress={true}
src="src"
theme={Object {}}
user={
Expand All @@ -69,15 +68,13 @@ exports[`plugins/Pluggable should match snapshot with no overridden component 1`
bsClass="popover"
id="user-profile-popover"
placement="right"
showEmailAddress={true}
theme={Object {}}
title="@undefined"
>
<div
className="popover right"
id="user-profile-popover"
role="tooltip"
showEmailAddress={true}
style={
Object {
"display": "block",
Expand Down
1 change: 0 additions & 1 deletion tests/plugins/pluggable.test.jsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,6 @@ describe('plugins/Pluggable', () => {
general: {
config: {
EnableWebrtc: 'true',
ShowEmailAddress: 'true',
},
},
},
Expand Down