Skip to content

Commit 924f9f6

Browse files
committed
Changed to show error and create button
1 parent 18df5c6 commit 924f9f6

File tree

2 files changed

+22
-10
lines changed

2 files changed

+22
-10
lines changed

app/components/Area/List/Area.js

Lines changed: 20 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,8 @@ class Area extends React.Component
1313
areas : null,
1414
error : ''
1515
};
16+
this.showError = this.showError.bind(this);
17+
this.listAreas = this.listAreas.bind(this);
1618
this.getAreas();
1719
}
1820

@@ -27,15 +29,11 @@ class Area extends React.Component
2729
});
2830
}
2931

30-
render() {
32+
listAreas() {
3133
if (this.state.error) {
32-
return (<Error error={this.state.error} />);
33-
}
34-
if (!this.state.areas) {
35-
return <div>Loading...</div>;
34+
return '';
3635
}
37-
const areaList = this.state.areas.map((area, key) => {
38-
let line = ((key % 2) ? 'is-success' : 'is-info');
36+
return this.state.areas.map((area, key) => {
3937
return (
4038
<tr key={key}>
4139
<td>
@@ -44,12 +42,26 @@ class Area extends React.Component
4442
</tr>
4543
);
4644
});
45+
}
46+
47+
showError() {
48+
if (this.state.error) {
49+
return (<Error error={this.state.error} />);
50+
}
51+
return '';
52+
}
53+
54+
render() {
55+
if (!this.state.areas && !this.state.error) {
56+
return <div>Loading...</div>;
57+
}
4758

4859
return (
4960
<section className="">
5061
<div className="container hello">
5162
<div className="level header">
5263
<div className="level-left">
64+
{ this.showError() }
5365
<h2 className="title is-2">Areas</h2>
5466
</div>
5567
<div className="level-right">
@@ -63,7 +75,7 @@ class Area extends React.Component
6375
</div>
6476
<table className="table">
6577
<tbody>
66-
{ areaList }
78+
{ this.listAreas() }
6779
</tbody>
6880
</table>
6981
</div>

tests/List.Area.test.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ describe('Test Area', () => {
3535
setTimeout(() => {
3636
try {
3737
component.update();
38-
expect(component.render().text()).toEqual('Area Not Found');
38+
expect(component.render().find('.is-danger').text()).toEqual('Area Not Found');
3939
done();
4040
} catch (e) {
4141
console.log(e);
@@ -62,7 +62,7 @@ describe('Test Area', () => {
6262
setTimeout(() => {
6363
try {
6464
component.update();
65-
expect(component.render().text()).toEqual('Error Found: Trying get area');
65+
expect(component.render().find('.is-danger').text()).toEqual('Error Found: Trying get area');
6666
done();
6767
} catch (e) {
6868
console.log(e);

0 commit comments

Comments
 (0)