SettingsControllerTest - Code Metrics - Inspection of "timezone select (#743)" - nextcloud/calendar - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 3ac45b...28dc8b )

by Georg

created 2018-03-05 14:47 UTC

SettingsControllerTest A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	296
Duplicated Lines	40.88 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

Metric	Value
wmc	23
lcom	1
cbo	1
dl	121
loc	296
rs	10
c	0
b	0
f	0

23 Methods

Rating	Name	Duplication	Size	Complexity
A	testSetFirstRun()	11	11	1
A	testSetFirstRunWithException()	0	12	1
A	testGetFirstRun()	0	12	1
B	setUp()	0	27	1
A	testSetViewWithAllowedView()	11	11	1
A	setViewWithAllowedViewDataProvider()	0	7	1
A	testSetViewWithForbiddenView()	7	7	1
A	testGetView()	0	12	1
A	testSetPopoverWithAllowedValue()	11	11	1
A	setPopoverWithAllowedValueDataProvider()	0	6	1
A	testSetPopoverWithForbiddenValue()	7	7	1
A	testSetPopoverWithConfigException()	0	12	1
A	testGetPopover()	0	12	1
A	testSetViewWithConfigException()	12	12	1
A	testGetViewWithConfigException()	12	12	1
A	testGetPopoverWithConfigException()	12	12	1
A	testGetFirstRunWithException()	12	12	1
A	testGetNotExistingConfig()	7	7	1
A	testSetNotExistingConfig()	7	7	1
A	testSetTimezone()	0	11	1
A	testSetTimezoneWithException()	0	12	1
A	testGetTimezone()	0	12	1
A	testGetTimezoneWithException()	12	12	1

How to fix Duplicated Code

<?php
/**
 * Calendar App
 *
 * @author Georg Ehrke
 * @copyright 2016 Georg Ehrke <[email protected]>
 *
 * This library is free software; you can redistribute it and/or
 * modify it under the terms of the GNU AFFERO GENERAL PUBLIC LICENSE
 * License as published by the Free Software Foundation; either
 * version 3 of the License, or any later version.
 *
 * This library is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 * GNU AFFERO GENERAL PUBLIC LICENSE for more details.
 *
 * You should have received a copy of the GNU Affero General Public
 * License along with this library.  If not, see <http://www.gnu.org/licenses/>.
 *
 */
namespace OCA\Calendar\Controller;

class SettingsControllerTest extends \PHPUnit_Framework_TestCase {

	private $appName;
	private $request;
	private $userSession;
	private $config;

	private $dummyUser;

	private $controller;

	public function setUp() {
		$this->appName = 'calendar';
		$this->request = $this->getMockBuilder('\OCP\IRequest')
			->disableOriginalConstructor()
			->getMock();
		$this->userSession = $this->getMockBuilder('OCP\IUserSession')
			->disableOriginalConstructor()
			->getMock();
		$this->config = $this->getMockBuilder('OCP\IConfig')
			->disableOriginalConstructor()
			->getMock();

		$this->dummyUser = $this->getMockBuilder('OCP\IUser')
			->disableOriginalConstructor()
			->getMock();

		$this->userSession->expects($this->once())
			->method('getUser')
			->will($this->returnValue($this->dummyUser));

		$this->dummyUser->expects($this->once())
			->method('getUID')
			->will($this->returnValue('user123'));

		$this->controller = new SettingsController($this->appName,
			$this->request, $this->userSession, $this->config);
	}

	/**
	 * @dataProvider setViewWithAllowedViewDataProvider
	 */
	public function testSetViewWithAllowedView($view) {

		$this->config->expects($this->once())
			->method('setUserValue')
			->with('user123', $this->appName, 'currentView', $view);

		$actual = $this->controller->setConfig('view', $view);

		$this->assertInstanceOf('OCP\AppFramework\Http\JSONResponse', $actual);
		$this->assertEquals([], $actual->getData());
		$this->assertEquals(200, $actual->getStatus());
	}

	public function setViewWithAllowedViewDataProvider() {
		return [
			['agendaDay'],
			['agendaWeek'],
			['month'],
		];
	}

	public function testSetViewWithForbiddenView() {

		$actual = $this->controller->setConfig('view','someForbiddenView');

		$this->assertInstanceOf('OCP\AppFramework\Http\JSONResponse', $actual);
		$this->assertEquals([], $actual->getData());
		$this->assertEquals(422, $actual->getStatus());
	}

	public function testSetViewWithConfigException() {

		$this->config->expects($this->once())
			->method('setUserValue')
			->with('user123', $this->appName, 'currentView', 'month')
			->will($this->throwException(new \Exception));

		$actual = $this->controller->setConfig('view', 'month');

		$this->assertInstanceOf('OCP\AppFramework\Http\JSONResponse', $actual);
		$this->assertEquals([], $actual->getData());
		$this->assertEquals(500, $actual->getStatus());
	}

	public function testGetView() {
		$this->config->expects($this->once())
			->method('getUserValue')
			->with('user123', $this->appName, 'currentView', 'month')
			->will($this->returnValue('agendaWeek'));

		$actual = $this->controller->getConfig('view');

		$this->assertInstanceOf('OCP\AppFramework\Http\JSONResponse', $actual);
		$this->assertEquals(['value' => 'agendaWeek'], $actual->getData());
		$this->assertEquals(200, $actual->getStatus());
	}

	public function testGetViewWithConfigException() {

		$this->config->expects($this->once())
			->method('getUserValue')
			->with('user123', $this->appName, 'currentView', 'month')
			->will($this->throwException(new \Exception));

		$actual = $this->controller->getConfig('view');

		$this->assertInstanceOf('OCP\AppFramework\Http\JSONResponse', $actual);
		$this->assertEquals([], $actual->getData());
		$this->assertEquals(500, $actual->getStatus());
	}

	/**
	 * @dataProvider setPopoverWithAllowedValueDataProvider
	 */
	public function testSetPopoverWithAllowedValue($value) {

		$this->config->expects($this->once())
			->method('setUserValue')
			->with('user123', $this->appName, 'skipPopover', $value);

		$actual = $this->controller->setConfig('skipPopover', $value);

		$this->assertInstanceOf('OCP\AppFramework\Http\JSONResponse', $actual);
		$this->assertEquals([], $actual->getData());
		$this->assertEquals(200, $actual->getStatus());
	}

	public function setPopoverWithAllowedValueDataProvider() {
		return [
			['yes'],
			['no']
		];
	}

	public function testSetPopoverWithForbiddenValue() {

		$actual = $this->controller->setConfig('skipPopover','someForbiddenValue');

		$this->assertInstanceOf('OCP\AppFramework\Http\JSONResponse', $actual);
		$this->assertEquals([], $actual->getData());
		$this->assertEquals(422, $actual->getStatus());
	}

	public function testSetPopoverWithConfigException() {
		$this->config->expects($this->once())
			->method('setUserValue')
			->with('user123', $this->appName, 'skipPopover', 'no')
			->will($this->throwException(new \Exception));

		$actual = $this->controller->setConfig('skipPopover', 'no');

		$this->assertInstanceOf('OCP\AppFramework\Http\JSONResponse', $actual);
		$this->assertEquals([], $actual->getData());
		$this->assertEquals(500, $actual->getStatus());
	}

	public function testGetPopover() {
		$this->config->expects($this->once())
			->method('getUserValue')
			->with('user123', $this->appName, 'skipPopover', 'no')
			->will($this->returnValue('agendaWeek'));

		$actual = $this->controller->getConfig('skipPopover');

		$this->assertInstanceOf('OCP\AppFramework\Http\JSONResponse', $actual);
		$this->assertEquals(['value' => 'agendaWeek'], $actual->getData());
		$this->assertEquals(200, $actual->getStatus());
	}

	public function testGetPopoverWithConfigException() {

		$this->config->expects($this->once())
			->method('getUserValue')
			->with('user123', $this->appName, 'skipPopover', 'no')
			->will($this->throwException(new \Exception));

		$actual = $this->controller->getConfig('skipPopover');

		$this->assertInstanceOf('OCP\AppFramework\Http\JSONResponse', $actual);
		$this->assertEquals([], $actual->getData());
		$this->assertEquals(500, $actual->getStatus());
	}

	public function testSetFirstRun() {

		$this->config->expects($this->once())
			->method('setUserValue')
			->with('user123', $this->appName, 'firstRun', 'no');

		$actual = $this->controller->setConfig('firstRun', 'some_random_ignored_value');

		$this->assertInstanceOf('OCP\AppFramework\Http\JSONResponse', $actual);
		$this->assertEquals([], $actual->getData());
		$this->assertEquals(200, $actual->getStatus());
	}

	public function testSetFirstRunWithException() {
		$this->config->expects($this->once())
			->method('setUserValue')
			->with('user123', $this->appName, 'firstRun', 'no')
			->will($this->throwException(new \Exception));

		$actual = $this->controller->setConfig('firstRun', 'some_random_ignored_value');

		$this->assertInstanceOf('OCP\AppFramework\Http\JSONResponse', $actual);
		$this->assertEquals([], $actual->getData());
		$this->assertEquals(500, $actual->getStatus());
	}

	public function testGetFirstRun() {
		$this->config->expects($this->once())
			->method('getUserValue')
			->with('user123', $this->appName, 'firstRun', 'yes')
			->will($this->returnValue('no'));

		$actual = $this->controller->getConfig('firstRun');

		$this->assertInstanceOf('OCP\AppFramework\Http\JSONResponse', $actual);
		$this->assertEquals(['value' => 'no'], $actual->getData());
		$this->assertEquals(200, $actual->getStatus());
	}

	public function testGetFirstRunWithException() {

		$this->config->expects($this->once())
			->method('getUserValue')
			->with('user123', $this->appName, 'firstRun', 'yes')
			->will($this->throwException(new \Exception));

		$actual = $this->controller->getConfig('firstRun');

		$this->assertInstanceOf('OCP\AppFramework\Http\JSONResponse', $actual);
		$this->assertEquals([], $actual->getData());
		$this->assertEquals(500, $actual->getStatus());
	}

	public function testSetTimezone() {
		$this->config->expects($this->once())
			->method('setUserValue')
			->with('user123', $this->appName, 'timezone', 'Europe/Berlin');

		$actual = $this->controller->setConfig('timezone', 'Europe/Berlin');

		$this->assertInstanceOf('OCP\AppFramework\Http\JSONResponse', $actual);
		$this->assertEquals([], $actual->getData());
		$this->assertEquals(200, $actual->getStatus());
	}

	public function testSetTimezoneWithException() {
		$this->config->expects($this->once())
			->method('setUserValue')
			->with('user123', $this->appName, 'timezone', 'Europe/Berlin')
			->will($this->throwException(new \Exception));

		$actual = $this->controller->setConfig('timezone', 'Europe/Berlin');

		$this->assertInstanceOf('OCP\AppFramework\Http\JSONResponse', $actual);
		$this->assertEquals([], $actual->getData());
		$this->assertEquals(500, $actual->getStatus());
	}

	public function testGetTimezone() {
		$this->config->expects($this->once())
			->method('getUserValue')
			->with('user123', $this->appName, 'timezone', 'automatic')
			->will($this->returnValue('Europe/Berlin'));

		$actual = $this->controller->getConfig('timezone');

		$this->assertInstanceOf('OCP\AppFramework\Http\JSONResponse', $actual);
		$this->assertEquals(['value' => 'Europe/Berlin'], $actual->getData());
		$this->assertEquals(200, $actual->getStatus());
	}

	public function testGetTimezoneWithException() {

		$this->config->expects($this->once())
			->method('getUserValue')
			->with('user123', $this->appName, 'timezone', 'automatic')
			->will($this->throwException(new \Exception));

		$actual = $this->controller->getConfig('timezone');

		$this->assertInstanceOf('OCP\AppFramework\Http\JSONResponse', $actual);
		$this->assertEquals([], $actual->getData());
		$this->assertEquals(500, $actual->getStatus());
	}

	public function testGetNotExistingConfig() {

		$actual = $this->controller->getConfig('foo');
		
		$this->assertInstanceOf('OCP\AppFramework\Http\JSONResponse', $actual);
		$this->assertEquals([], $actual->getData());
		$this->assertEquals(400, $actual->getStatus());
	}

	public function testSetNotExistingConfig() {

		$actual = $this->controller->setConfig('foo', 'bar');

		$this->assertInstanceOf('OCP\AppFramework\Http\JSONResponse', $actual);
		$this->assertEquals([], $actual->getData());
		$this->assertEquals(400, $actual->getStatus());
	}
}


1		<?php
2		/**
3		* Calendar App
4		*
5		* @author Georg Ehrke
6		* @copyright 2016 Georg Ehrke <[email protected]>
7		*
8		* This library is free software; you can redistribute it and/or
9		* modify it under the terms of the GNU AFFERO GENERAL PUBLIC LICENSE
10		* License as published by the Free Software Foundation; either
11		* version 3 of the License, or any later version.
12		*
13		* This library is distributed in the hope that it will be useful,
14		* but WITHOUT ANY WARRANTY; without even the implied warranty of
15		* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
16		* GNU AFFERO GENERAL PUBLIC LICENSE for more details.
17		*
18		* You should have received a copy of the GNU Affero General Public
19		* License along with this library. If not, see <http://www.gnu.org/licenses/>.
20		*
21		*/
22		namespace OCA\Calendar\Controller;
23
24		class SettingsControllerTest extends \PHPUnit_Framework_TestCase {
25
26		private $appName;
27		private $request;
28		private $userSession;
29		private $config;
30
31		private $dummyUser;
32
33		private $controller;
34
35		public function setUp() {
36		$this->appName = 'calendar';
37		$this->request = $this->getMockBuilder('\OCP\IRequest')
38		->disableOriginalConstructor()
39		->getMock();
40		$this->userSession = $this->getMockBuilder('OCP\IUserSession')
41		->disableOriginalConstructor()
42		->getMock();
43		$this->config = $this->getMockBuilder('OCP\IConfig')
44		->disableOriginalConstructor()
45		->getMock();
46
47		$this->dummyUser = $this->getMockBuilder('OCP\IUser')
48		->disableOriginalConstructor()
49		->getMock();
50
51		$this->userSession->expects($this->once())
52		->method('getUser')
53		->will($this->returnValue($this->dummyUser));
54
55		$this->dummyUser->expects($this->once())
56		->method('getUID')
57		->will($this->returnValue('user123'));
58
59		$this->controller = new SettingsController($this->appName,
60		$this->request, $this->userSession, $this->config);
61		}
62
63		/**
64		* @dataProvider setViewWithAllowedViewDataProvider
65		*/
66	View Code Duplication	public function testSetViewWithAllowedView($view) {
		0 ignored issues – show Duplication introduced 2017-08-01 00:26 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
67		$this->config->expects($this->once())
68		->method('setUserValue')
69		->with('user123', $this->appName, 'currentView', $view);
70
71		$actual = $this->controller->setConfig('view', $view);
72
73		$this->assertInstanceOf('OCP\AppFramework\Http\JSONResponse', $actual);
74		$this->assertEquals([], $actual->getData());
75		$this->assertEquals(200, $actual->getStatus());
76		}
77
78		public function setViewWithAllowedViewDataProvider() {
79		return [
80		['agendaDay'],
81		['agendaWeek'],
82		['month'],
83		];
84		}
85
86	View Code Duplication	public function testSetViewWithForbiddenView() {
		0 ignored issues – show Duplication introduced 2017-08-01 00:26 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
87		$actual = $this->controller->setConfig('view','someForbiddenView');
88
89		$this->assertInstanceOf('OCP\AppFramework\Http\JSONResponse', $actual);
90		$this->assertEquals([], $actual->getData());
91		$this->assertEquals(422, $actual->getStatus());
92		}
93
94	View Code Duplication	public function testSetViewWithConfigException() {
		0 ignored issues – show Duplication introduced 2018-02-26 11:30 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
95		$this->config->expects($this->once())
96		->method('setUserValue')
97		->with('user123', $this->appName, 'currentView', 'month')
98		->will($this->throwException(new \Exception));
99
100		$actual = $this->controller->setConfig('view', 'month');
101
102		$this->assertInstanceOf('OCP\AppFramework\Http\JSONResponse', $actual);
103		$this->assertEquals([], $actual->getData());
104		$this->assertEquals(500, $actual->getStatus());
105		}
106
107		public function testGetView() {
108		$this->config->expects($this->once())
109		->method('getUserValue')
110		->with('user123', $this->appName, 'currentView', 'month')
111		->will($this->returnValue('agendaWeek'));
112
113		$actual = $this->controller->getConfig('view');
114
115		$this->assertInstanceOf('OCP\AppFramework\Http\JSONResponse', $actual);
116		$this->assertEquals(['value' => 'agendaWeek'], $actual->getData());
117		$this->assertEquals(200, $actual->getStatus());
118		}
119
120	View Code Duplication	public function testGetViewWithConfigException() {
		0 ignored issues – show Duplication introduced 2018-02-26 11:30 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
121		$this->config->expects($this->once())
122		->method('getUserValue')
123		->with('user123', $this->appName, 'currentView', 'month')
124		->will($this->throwException(new \Exception));
125
126		$actual = $this->controller->getConfig('view');
127
128		$this->assertInstanceOf('OCP\AppFramework\Http\JSONResponse', $actual);
129		$this->assertEquals([], $actual->getData());
130		$this->assertEquals(500, $actual->getStatus());
131		}
132
133		/**
134		* @dataProvider setPopoverWithAllowedValueDataProvider
135		*/
136	View Code Duplication	public function testSetPopoverWithAllowedValue($value) {
		0 ignored issues – show Duplication introduced 2017-08-01 00:26 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
137		$this->config->expects($this->once())
138		->method('setUserValue')
139		->with('user123', $this->appName, 'skipPopover', $value);
140
141		$actual = $this->controller->setConfig('skipPopover', $value);
142
143		$this->assertInstanceOf('OCP\AppFramework\Http\JSONResponse', $actual);
144		$this->assertEquals([], $actual->getData());
145		$this->assertEquals(200, $actual->getStatus());
146		}
147
148		public function setPopoverWithAllowedValueDataProvider() {
149		return [
150		['yes'],
151		['no']
152		];
153		}
154
155	View Code Duplication	public function testSetPopoverWithForbiddenValue() {
		0 ignored issues – show Duplication introduced 2017-08-01 00:26 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
156		$actual = $this->controller->setConfig('skipPopover','someForbiddenValue');
157
158		$this->assertInstanceOf('OCP\AppFramework\Http\JSONResponse', $actual);
159		$this->assertEquals([], $actual->getData());
160		$this->assertEquals(422, $actual->getStatus());
161		}
162
163		public function testSetPopoverWithConfigException() {
164		$this->config->expects($this->once())
165		->method('setUserValue')
166		->with('user123', $this->appName, 'skipPopover', 'no')
167		->will($this->throwException(new \Exception));
168
169		$actual = $this->controller->setConfig('skipPopover', 'no');
170
171		$this->assertInstanceOf('OCP\AppFramework\Http\JSONResponse', $actual);
172		$this->assertEquals([], $actual->getData());
173		$this->assertEquals(500, $actual->getStatus());
174		}
175
176		public function testGetPopover() {
177		$this->config->expects($this->once())
178		->method('getUserValue')
179		->with('user123', $this->appName, 'skipPopover', 'no')
180		->will($this->returnValue('agendaWeek'));
181
182		$actual = $this->controller->getConfig('skipPopover');
183
184		$this->assertInstanceOf('OCP\AppFramework\Http\JSONResponse', $actual);
185		$this->assertEquals(['value' => 'agendaWeek'], $actual->getData());
186		$this->assertEquals(200, $actual->getStatus());
187		}
188
189	View Code Duplication	public function testGetPopoverWithConfigException() {
		0 ignored issues – show Duplication introduced 2018-02-26 11:30 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
190		$this->config->expects($this->once())
191		->method('getUserValue')
192		->with('user123', $this->appName, 'skipPopover', 'no')
193		->will($this->throwException(new \Exception));
194
195		$actual = $this->controller->getConfig('skipPopover');
196
197		$this->assertInstanceOf('OCP\AppFramework\Http\JSONResponse', $actual);
198		$this->assertEquals([], $actual->getData());
199		$this->assertEquals(500, $actual->getStatus());
200		}
201
202	View Code Duplication	public function testSetFirstRun() {
		0 ignored issues – show Duplication introduced 2017-08-01 00:26 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
203		$this->config->expects($this->once())
204		->method('setUserValue')
205		->with('user123', $this->appName, 'firstRun', 'no');
206
207		$actual = $this->controller->setConfig('firstRun', 'some_random_ignored_value');
208
209		$this->assertInstanceOf('OCP\AppFramework\Http\JSONResponse', $actual);
210		$this->assertEquals([], $actual->getData());
211		$this->assertEquals(200, $actual->getStatus());
212		}
213
214		public function testSetFirstRunWithException() {
215		$this->config->expects($this->once())
216		->method('setUserValue')
217		->with('user123', $this->appName, 'firstRun', 'no')
218		->will($this->throwException(new \Exception));
219
220		$actual = $this->controller->setConfig('firstRun', 'some_random_ignored_value');
221
222		$this->assertInstanceOf('OCP\AppFramework\Http\JSONResponse', $actual);
223		$this->assertEquals([], $actual->getData());
224		$this->assertEquals(500, $actual->getStatus());
225		}
226
227		public function testGetFirstRun() {
228		$this->config->expects($this->once())
229		->method('getUserValue')
230		->with('user123', $this->appName, 'firstRun', 'yes')
231		->will($this->returnValue('no'));
232
233		$actual = $this->controller->getConfig('firstRun');
234
235		$this->assertInstanceOf('OCP\AppFramework\Http\JSONResponse', $actual);
236		$this->assertEquals(['value' => 'no'], $actual->getData());
237		$this->assertEquals(200, $actual->getStatus());
238		}
239
240	View Code Duplication	public function testGetFirstRunWithException() {
		0 ignored issues – show Duplication introduced 2018-02-26 11:30 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
241		$this->config->expects($this->once())
242		->method('getUserValue')
243		->with('user123', $this->appName, 'firstRun', 'yes')
244		->will($this->throwException(new \Exception));
245
246		$actual = $this->controller->getConfig('firstRun');
247
248		$this->assertInstanceOf('OCP\AppFramework\Http\JSONResponse', $actual);
249		$this->assertEquals([], $actual->getData());
250		$this->assertEquals(500, $actual->getStatus());
251		}
252
253		public function testSetTimezone() {
254		$this->config->expects($this->once())
255		->method('setUserValue')
256		->with('user123', $this->appName, 'timezone', 'Europe/Berlin');
257
258		$actual = $this->controller->setConfig('timezone', 'Europe/Berlin');
259
260		$this->assertInstanceOf('OCP\AppFramework\Http\JSONResponse', $actual);
261		$this->assertEquals([], $actual->getData());
262		$this->assertEquals(200, $actual->getStatus());
263		}
264
265		public function testSetTimezoneWithException() {
266		$this->config->expects($this->once())
267		->method('setUserValue')
268		->with('user123', $this->appName, 'timezone', 'Europe/Berlin')
269		->will($this->throwException(new \Exception));
270
271		$actual = $this->controller->setConfig('timezone', 'Europe/Berlin');
272
273		$this->assertInstanceOf('OCP\AppFramework\Http\JSONResponse', $actual);
274		$this->assertEquals([], $actual->getData());
275		$this->assertEquals(500, $actual->getStatus());
276		}
277
278		public function testGetTimezone() {
279		$this->config->expects($this->once())
280		->method('getUserValue')
281		->with('user123', $this->appName, 'timezone', 'automatic')
282		->will($this->returnValue('Europe/Berlin'));
283
284		$actual = $this->controller->getConfig('timezone');
285
286		$this->assertInstanceOf('OCP\AppFramework\Http\JSONResponse', $actual);
287		$this->assertEquals(['value' => 'Europe/Berlin'], $actual->getData());
288		$this->assertEquals(200, $actual->getStatus());
289		}
290
291	View Code Duplication	public function testGetTimezoneWithException() {
		0 ignored issues – show Duplication introduced 2018-02-26 11:30 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
292		$this->config->expects($this->once())
293		->method('getUserValue')
294		->with('user123', $this->appName, 'timezone', 'automatic')
295		->will($this->throwException(new \Exception));
296
297		$actual = $this->controller->getConfig('timezone');
298
299		$this->assertInstanceOf('OCP\AppFramework\Http\JSONResponse', $actual);
300		$this->assertEquals([], $actual->getData());
301		$this->assertEquals(500, $actual->getStatus());
302		}
303
304	View Code Duplication	public function testGetNotExistingConfig() {
		0 ignored issues – show Duplication introduced 2017-04-18 11:02 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
305		$actual = $this->controller->getConfig('foo');
306
307		$this->assertInstanceOf('OCP\AppFramework\Http\JSONResponse', $actual);
308		$this->assertEquals([], $actual->getData());
309		$this->assertEquals(400, $actual->getStatus());
310		}
311
312	View Code Duplication	public function testSetNotExistingConfig() {
		0 ignored issues – show Duplication introduced 2017-04-18 11:02 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
313		$actual = $this->controller->setConfig('foo', 'bar');
314
315		$this->assertInstanceOf('OCP\AppFramework\Http\JSONResponse', $actual);
316		$this->assertEquals([], $actual->getData());
317		$this->assertEquals(400, $actual->getStatus());
318		}
319		}
320

nextcloud / calendar

Push — master ( 3ac45b...28dc8b )

SettingsControllerTest A

Complexity

Size/Duplication

Coupling/Cohesion

Importance

23 Methods

How to fix Duplicated Code

Duplicated Code

Duplication Side-by-Side

Filter issues like