Passio Practice Exercise implemented

[shreyasgosavi]

pull request

Hello, raising PR for paasio exercise implementation. Issue ID : #1813

---

Reviewer Resources:

Track Policies

[kahgoh]

This file should just contain enough of the stub for students to start. Have a look at the stub for the Poker for example. It only contains the bare methods for students to fill.

When the student starts the Paasio exercise, the student will start with the contents of this file. They should not start with the solution.

[kahgoh]

Don't catch the exception in the tests, othewise the tests will pass if an exception is thrown. Instead, let propagate the exception so that JUnit also sees the exception.

Suggested change

	void checkReadOperationCount() {
	try (Paasio customFileReader = new Paasio(this.dataInputStream, this.dataOutputStream)) {
	customFileReader.read();
	customFileReader.read();
	customFileReader.read();
	assertThat(customFileReader.getReadOperationCount()).isEqualTo(3);
	} catch (IOException ioException) {
	System.out.println("Exception occured");
	}
	}
	void checkReadOperationCount() throw Exception {
	try (Paasio customFileReader = new Paasio(this.dataInputStream, this.dataOutputStream)) {
	customFileReader.read();
	customFileReader.read();
	customFileReader.read();
	assertThat(customFileReader.getReadOperationCount()).isEqualTo(3);
	}
	}

[kahgoh]

Please don't use * imports, as per other tests (poker for example).

[kahgoh]

I think it would be interesting to assert on the contents readByteData.

[kahgoh]

In general, only the first test should be "enabled" in practice exercises. Students enable the test as they work through the exercise.

Suggested change

	@Test
	@Disabled("Remove to run test")
	@Test

[kahgoh]

Please remove all System.out.println.

Suggested change

	System.out.println("haha ");
	System.out.println(fileOperations.getReadOperationCount());
	System.out.println("haha ");
	System.out.println(fileOperations.getReadOperationCount());

[kahgoh]

This really shouldn't be a one (it's not that easy). Perhaps we could use the Go track as a guide - they have used 4 as the difficulty.

[kahgoh]

Please don't use * imports. For example:

Suggested change

	import java.io.*;
	import java.io.Closeable;
	import java.io.InputStream;
	import java.io.OutputStream;

[kahgoh]

Since every test creates and uses a Paasio object, we could also use a Passio field and set it in setUpTest and then close it after every test in a @AfterEach method.

[kahgoh]

Suggested change

	public void setUPTest(TestInfo testInfo) {
	public void setUpTest(TestInfo testInfo) {

[shreyasgosavi]

Thanks for time and that's really good explanation will do it 🤘