use binder from context to make extensions it testable

This commit is contained in:
Sebastian Sdorra
2020-02-19 09:48:46 +01:00
parent 041a999a01
commit f23b464524
3 changed files with 68 additions and 75 deletions

View File

@@ -1,25 +1,26 @@
import React, {FC} from "react"; import React, { FC } from "react";
import { binder } from "@scm-manager/ui-extensions";
import { Image } from ".."; import { Image } from "..";
import { Person } from "./Avatar"; import { Person } from "./Avatar";
import { EXTENSION_POINT } from "./Avatar"; import { EXTENSION_POINT } from "./Avatar";
import { useBinder } from "@scm-manager/ui-extensions";
type Props = { type Props = {
person: Person; person: Person;
representation?: "rounded" | "rounded-border"; representation?: "rounded" | "rounded-border";
}; };
const AvatarImage:FC<Props> = ({person, representation = "rounded-border"}) => { const AvatarImage: FC<Props> = ({ person, representation = "rounded-border" }) => {
const avatarFactory = binder.getExtension(EXTENSION_POINT); const binder = useBinder();
if (avatarFactory) { const avatarFactory = binder.getExtension(EXTENSION_POINT);
const avatar = avatarFactory(person); if (avatarFactory) {
const avatar = avatarFactory(person);
const className = representation === "rounded" ? "is-rounded" : "has-rounded-border"; const className = representation === "rounded" ? "is-rounded" : "has-rounded-border";
return <Image className={className} src={avatar} alt={person.name} />; return <Image className={className} src={avatar} alt={person.name} />;
} }
return null; return null;
}; };
export default AvatarImage; export default AvatarImage;

View File

@@ -1,18 +1,13 @@
import React, { Component, ReactNode } from "react"; import React, { FC } from "react";
import { binder } from "@scm-manager/ui-extensions"; import { useBinder } from "@scm-manager/ui-extensions";
import { EXTENSION_POINT } from "./Avatar"; import { EXTENSION_POINT } from "./Avatar";
type Props = { const AvatarWrapper: FC = ({ children }) => {
children: ReactNode; const binder = useBinder();
if (binder.hasExtension(EXTENSION_POINT)) {
return <>{children}</>;
}
return null;
}; };
class AvatarWrapper extends Component<Props> {
render() {
if (binder.hasExtension(EXTENSION_POINT)) {
return <>{this.props.children}</>;
}
return null;
}
}
export default AvatarWrapper; export default AvatarWrapper;

View File

@@ -1,7 +1,7 @@
import React from "react"; import React, { FC } from "react";
import { Me, Links } from "@scm-manager/ui-types"; import { Me, Links } from "@scm-manager/ui-types";
import { Link } from "react-router-dom"; import { Link } from "react-router-dom";
import { binder } from "@scm-manager/ui-extensions"; import { useBinder } from "@scm-manager/ui-extensions";
import { AvatarWrapper, AvatarImage } from "../avatar"; import { AvatarWrapper, AvatarImage } from "../avatar";
type Props = { type Props = {
@@ -10,56 +10,53 @@ type Props = {
links: Links; links: Links;
}; };
class Footer extends React.Component<Props> { const Footer: FC<Props> = ({ me, version, links }) => {
render() { const binder = useBinder();
const { me, version, links } = this.props; if (!me) {
if (!me) { return null;
return "";
}
const extensionProps = { me, links };
const extensions = binder.getExtensions("footer.links", extensionProps);
return (
<footer className="footer">
<div className="container is-centered has-text-centered ">
<p className="is-size-6 columns is-centered">
<AvatarWrapper>
<figure className="media-left">
<p className="image is-24x24 is-rounded">
<AvatarImage person={me} representation="rounded" />
</p>
</figure>
</AvatarWrapper>
<Link to={"/me"}>{me.displayName}</Link>
</p>
<hr className={"has-background-grey-lighter"} />
<p>
<a href="https://www.scm-manager.org/" target="_blank">
SCM-Manager {version}
</a>
{extensions.map(Ext => (
<>
{" "}
| <Ext {...extensionProps} />
</>
))}
</p>
<br />
<p>
Powered by{" "}
<a target="_blank" href="https://cloudogu.com/">
Cloudogu GmbH
</a>{" "}
| Learn more about{" "}
<a href="https://www.scm-manager.org/" target="_blank">
SCM-Manager
</a>
</p>
</div>
</footer>
);
} }
} const extensionProps = { me, links };
const extensions = binder.getExtensions("footer.links", extensionProps);
return (
<footer className="footer">
<div className="container is-centered has-text-centered ">
<p className="is-size-6 columns is-centered">
<AvatarWrapper>
<figure className="media-left">
<p className="image is-24x24 is-rounded">
<AvatarImage person={me} representation="rounded" />
</p>
</figure>
</AvatarWrapper>
<Link to={"/me"}>{me.displayName}</Link>
</p>
<hr className={"has-background-grey-lighter"} />
<p>
<a href="https://www.scm-manager.org/" target="_blank">
SCM-Manager {version}
</a>
{extensions.map(Ext => (
<>
{" "}
| <Ext {...extensionProps} />
</>
))}
</p>
<br />
<p>
Powered by{" "}
<a target="_blank" href="https://cloudogu.com/">
Cloudogu GmbH
</a>{" "}
| Learn more about{" "}
<a href="https://www.scm-manager.org/" target="_blank">
SCM-Manager
</a>
</p>
</div>
</footer>
);
};
export default Footer; export default Footer;