Passio Practice Exercise implemented by shreyasgosavi · Pull Request #2970 · exercism/java · GitHub

pull request

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

Reviewer Resources:

…required files from the resources template to the exercise

… covered | Sample Implementation for some of the read methods is done

…mentation in FileOperations

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.

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);
          
                    }
          
                }

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

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

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

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());

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.

Please don't use * imports. For example:

Suggested change

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

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.

Suggested change

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

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

Heyyy I have made the requested changes could you please verify that

As mentioned earlier, please don't use * for import

Same goes for this!

Every test except the first one should have @Disabled("Remove to run test")

Could you please move this exercise up with the other ones that have a difficulty of 4, and make sure it stays in alphabetical order within that section?

I think this could be simplified to simply calling passioReaderWriter.close() and might allow you to remove getInputStream and getOutputStream methods from the Passio class.

There seems to be extra spacing here.

Suggested change

I wasn't sure - why do we need to subtract 2 here?

I think this fileData name came from an earlier iteration where we had a distinction between file and network data. However, we no longer have this distinction, so I'd suggest renaming it. Perhaps just data?

Suggested change

      
                    String finalVAlue = new String(dataRead, 0, helloArray.length + bytesRead, StandardCharsets.UTF_8);
          
                    String finalValue = new String(dataRead, 0, helloArray.length + bytesRead, StandardCharsets.UTF_8);

I think we should also assert the contents byteData as it should write only 10 bytes to the array. This should be less than the available content?

Should there also be a test for when the length (or the "10") is more than what is available?

I think we are also missing tests that assert on the byte[] that is returned by the readNBytes method.

It is more idiomatic to use screaming snake case for constants in Java. Could you please also move this to just under the public class PaasioTest { line? (its more idiomatic that way)

Suggested change

      
                private static final String MESSAGECONSTANT = "This is additional Content.";
          
                private static final String MESSAGE_CONSTANT = "This is additional Content.";

I think this is missing asserting on the contents that are written to the dataOutputStream.

Similar to another comment, this isn't "file" specific, so suggest renaming the variable.

shreyasgosavi added 19 commits May 23, 2025 16:02

Setting up paasio exercise | Added paasio in settings.gradle | Added …

a912d0c

…required files from the resources template to the exercise

Added Sample code to avoid Build failure

d81eb05

Paasio Code begins | Started Code structure set-up

bc2a2cd

Developing with TDD strategy | Test-cases for file-reading operations…

c83978c

… covered | Sample Implementation for some of the read methods is done

Test cases for Write Operations for file handling written

52e1794

Sample Code added for File-Operations

3507f1c

FileOperation Changes added

4a94e3c

Adding Generic Input-Output Operation class | Referring to that imple…

d1d9917

…mentation in FileOperations

Adding tags to the File Operation Test Cases

4c67d9b

Test Cases for SOcket Opearations added. along with its implementations

9706a9f

Another test case to verify data written in output-stream

8c3aaf5

Changed the implementation to more generic code.

0aeb078

Code formatted

71e0867

Un-necessary test-cases consolidated and shortened

b934489

Merge branch 'paasio-exercise'

b8ba510

Sonar Qube changes added | Changes in test cases -- corrected arguments

e07be74

Merge branch 'paasio-exercise'

03ee525

Changes synced with reference code

47ef869

Code formatted

fbf5c8d

shreyasgosavi marked this pull request as draft July 5, 2025 05:17

kahgoh changed the title Passi Practice Exercise implemented Passio Practice Exercise implemented Jul 5, 2025

kahgoh requested changes Jul 9, 2025

View reviewed changes

shreyasgosavi and others added 2 commits August 6, 2025 01:16

Changes suggested by kahgoh were made

3e71a5b

Merge branch 'exercism:main' into main

251ef7e

shreyasgosavi marked this pull request as ready for review August 5, 2025 19:58

jagdish-15 requested changes Aug 6, 2025

View reviewed changes

kahgoh reviewed Aug 6, 2025

View reviewed changes

Merge branch 'exercism:main' into main

e24a11d

+              import org.junit.jupiter.api.*;
+              import java.io.*;
+              import java.nio.charset.StandardCharsets;
+              import java.util.Arrays;
+              import static org.assertj.core.api.Assertions.*;

+                      "uuid": "81177978-2ed3-47c2-a6a0-fef316d94c0b",
+                      "practices": [],
+                      "prerequisites": [],
+                      "difficulty": 1

+                      if (!Objects.isNull(this.paasioReaderWriter.getInputStream())) {
+                          this.paasioReaderWriter.getInputStream().close();
+                      }
+                      if (!Objects.isNull(this.paasioReaderWriter.getOutputStream())) {
+                          this.paasioReaderWriter.getOutputStream().close();
+                      }

+                          byte[] readByteData = new byte[5];
+                          fileOperations.read(readByteData);
+                          assertThat(fileOperations.getBytesRead()).isEqualTo(9);

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

+              public class PaasioTest {
+                  private ByteArrayInputStream dataInputStream;
+                  private ByteArrayOutputStream dataOutputStream;

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

+                      assertThat(paasioReaderWriter.getBytesRead()).isEqualTo(4);
+                  }
+                  @Test

                       ],
                       "difficulty": 10
+                    },
+                    {

+                      byte[] initialMessage = "Hello! ".getBytes();
+                      byte[] messageArray = Arrays.copyOf(initialMessage, 50);
+                      int length = messageArray.length - initialMessage.length - 2;

+                  @Disabled("Remove to run test")
+                  void checkByteValueReadFromTheFile() throws IOException {
+                      byte[] fileData = new byte[100];

+                      byte[] dataRead = Arrays.copyOf(helloArray, 100);
+                      int bytesRead = paasioReaderWriter.read(dataRead, helloArray.length, 40);
+                      String finalVAlue = new String(dataRead, 0, helloArray.length + bytesRead, StandardCharsets.UTF_8);

	String finalVAlue = new String(dataRead, 0, helloArray.length + bytesRead, StandardCharsets.UTF_8);
	String finalValue = new String(dataRead, 0, helloArray.length + bytesRead, StandardCharsets.UTF_8);

		@@ -0,0 +1,140 @@
		import java.io.*;

		@@ -0,0 +1,98 @@
		import java.io.*;

		@@ -0,0 +1,149 @@
		import java.io.*;

+                      byte[] byteData = new byte[50];
+                      int bytesRead = paasioReaderWriter.read(byteData, 0, 10);
+                      assertThat(bytesRead).isEqualTo(10);

+                  void validateReadNBytesReadOperationCount() throws IOException {
+                      paasioReaderWriter.readNBytes(5);
+                      paasioReaderWriter.readNBytes(5);

	private static final String MESSAGECONSTANT = "This is additional Content.";
	private static final String MESSAGE_CONSTANT = "This is additional Content.";

+                      paasioReaderWriter.write(writeFileContent);
+                      paasioReaderWriter.write(writeFileContent);
+                      paasioReaderWriter.write('s');

+                  @Disabled("Remove to run test")
+                  void verifyBytesWrittenFromOffsetIntoTheOutputStreamAlongWithTheCount() throws IOException {
+                      String fileContentToBeWritten = "   This is additional Content.";

Passio Practice Exercise implemented by shreyasgosavi · Pull Request #2970 · exercism/java · GitHub

Uh oh!

Uh oh!

Passio Practice Exercise implemented - #2970

Passio Practice Exercise implemented#2970shreyasgosavi wants to merge 22 commits intoexercism:mainexercism/java:mainfrom shreyasgosavi:mainshreyasgosavi/exercism-java:mainCopy head branch name to clipboard

Conversation

shreyasgosavi commented Jul 5, 2025

pull request

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

shreyasgosavi commented Jul 12, 2025

Uh oh!

shreyasgosavi commented Aug 5, 2025

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Passio Practice Exercise implemented#2970
shreyasgosavi wants to merge 22 commits into
exercism:mainfrom
shreyasgosavi:main