Moved getBranchFromMatch to ui-components

This commit is contained in:
Philipp Czora
2018-10-17 17:11:21 +02:00
parent 5871bf052f
commit 8ff19a3d2f
5 changed files with 14 additions and 12 deletions

View File

@@ -21,7 +21,8 @@ import ChangesetList from "../components/changesets/ChangesetList";
import {
ErrorNotification,
LinkPaginator,
Loading
Loading,
getPageFromMatch
} from "@scm-manager/ui-components";
import { compose } from "redux";
@@ -94,14 +95,6 @@ const mapDispatchToProps = dispatch => {
};
};
export function getPageFromMatch(match: any) {
let page = parseInt(match.params.page, 10);
if (isNaN(page) || !page) {
page = 1;
}
return page;
}
const mapStateToProps = (state: any, ownProps: Props) => {
const { repository, branch, match } = ownProps;
const changesets = getChangesets(state, repository, branch);

View File

@@ -1,27 +0,0 @@
// @flow
import { getPageFromMatch } from "./Changesets";
describe("tests for getPageFromMatch", () => {
function createMatch(page: string) {
return {
params: {
page
}
};
}
it("should return 1 for NaN", () => {
const match = createMatch("any");
expect(getPageFromMatch(match)).toBe(1);
});
it("should return 1 for 0", () => {
const match = createMatch("0");
expect(getPageFromMatch(match)).toBe(1);
});
it("should return the given number", () => {
const match = createMatch("42");
expect(getPageFromMatch(match)).toBe(42);
});
});