Writing_On_GitHub_Persist_Client_Test

Complexity

Total Complexity

2

Size/Duplication

Total Lines	16
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	1

2 Methods

Rating	Name	Duplication	Size	Complexity
A	setUp()	0	4	1
A	test_should_delete_file()	0	5	1

<?php

/**
 * @group api
 */
class Writing_On_GitHub_Persist_Client_Test extends Writing_On_GitHub_Base_Client_Test {

    public function setUp() {

The function name setUp is in camel caps, but expected set_up instead as per the coding standard.

        parent::setUp();

        $this->persist = new Writing_On_GitHub_Persist_Client( $this->app );

It seems like $this->app can also be of type Mockery\Mock; however, parameter $app of Writing_On_GitHub_Persist_Client::__construct() does only seem to accept Writing_On_GitHub, maybe add an additional type check?

        // $this->commit
        //  ->shouldReceive( 'tree' )
        //  ->andReturn( $this->tree );
    }

    public function test_should_delete_file() {
        // $this->tree->shouldReceive();

        $this->set_delete_contents( true, '_posts_test.md' );
        $this->persist->delete_file( '_posts_test.md', 'mysha', 'this is message for delete file' );
    }

    // public function test_should_not_add_commit_if_no_change() {
    //  $this->tree
    //      ->shouldReceive( 'is_changed' )
    //      ->once()
    //      ->andReturn( false );

    //  $this->assertWPError( $error = $this->persist->commit( $this->commit ) );
    //  $this->assertSame( 'no_commit', $error->get_error_code() );
    // }

    // public function test_should_fail_if_create_tree_fails() {
    //  $this->tree
    //      ->shouldReceive( 'is_changed' )
    //      ->once()
    //      ->andReturn( true );
    //  $this->tree
    //      ->shouldReceive( 'to_body' )
    //      ->once()
    //      ->andReturn(
    //          array(
    //              'tree' => array(
    //                  array(
    //                      'path'    => '_posts/2015-10-23-new-post.md',
    //                      'type'    => 'blob',
    //                      'content' => 'Post content 1',
    //                      'mode'    => '100644',
    //                  )
    //              )
    //          )
    //      );
    //  $this->set_post_trees( false );

    //  $this->assertWPError( $error = $this->persist->commit( $this->commit ) );
    //  $this->assertSame( '404_not_found', $error->get_error_code() );
    // }

    // public function test_should_fail_if_create_commit_fails() {
    //  $this->tree
    //      ->shouldReceive( 'is_changed' )
    //      ->once()
    //      ->andReturn( true );
    //  $this->tree
    //      ->shouldReceive( 'to_body' )
    //      ->once()
    //      ->andReturn(
    //          array(
    //              'tree' => array(
    //                  array(
    //                      'path'    => '_posts/2015-10-23-new-post.md',
    //                      'type'    => 'blob',
    //                      'content' => 'Post content 1',
    //                      'mode'    => '100644',
    //                  )
    //              )
    //          )
    //      );
    //  $this->tree
    //      ->shouldReceive( 'set_sha' )
    //      ->once()
    //      ->with( 'c0d2cb90b51826096c826b61a0e74d2c973d7ad8' );
    //  $this->commit
    //      ->shouldReceive( 'to_body' )
    //      ->once()
    //      ->andReturn( array(
    //          'tree'    => 'c0d2cb90b51826096c826b61a0e74d2c973d7ad8',
    //          'message' => 'Commit message',
    //          'parents' => array( 'c0d2cb90b51826096c826b61a0e74d2c973d7ad8' )
    //      ) );
    //  $this->set_post_trees( true );
    //  $this->set_post_commits( false );

    //  $this->assertWPError( $error = $this->persist->commit( $this->commit ) );
    //  $this->assertSame( '404_not_found', $error->get_error_code() );
    // }

    // public function test_should_fail_if_update_master_fails() {
    //  $this->tree
    //      ->shouldReceive( 'is_changed' )
    //      ->once()
    //      ->andReturn( true );
    //  $this->tree
    //      ->shouldReceive( 'to_body' )
    //      ->once()
    //      ->andReturn(
    //          array(
    //              'tree' => array(
    //                  array(
    //                      'path'    => '_posts/2015-10-23-new-post.md',
    //                      'type'    => 'blob',
    //                      'content' => 'Post content 1',
    //                      'mode'    => '100644',
    //                  )
    //              )
    //          )
    //      );
    //  $this->tree
    //      ->shouldReceive( 'set_sha' )
    //      ->once()
    //      ->with( 'c0d2cb90b51826096c826b61a0e74d2c973d7ad8' );
    //  $this->commit
    //      ->shouldReceive( 'to_body' )
    //      ->once()
    //      ->andReturn( array(
    //          'tree'    => 'c0d2cb90b51826096c826b61a0e74d2c973d7ad8',
    //          'message' => 'Commit message',
    //          'parents' => array( 'c0d2cb90b51826096c826b61a0e74d2c973d7ad8' )
    //      ) );
    //  $this->set_post_trees( true );
    //  $this->set_post_commits( true );
    //  $this->set_patch_refs_heads_master( false );

    //  $this->assertWPError( $error = $this->persist->commit( $this->commit ) );
    //  $this->assertSame( '404_not_found', $error->get_error_code() );
    // }

    // public function test_should_create_anonymous_commit() {
    //  $this->tree
    //      ->shouldReceive( 'is_changed' )
    //      ->once()
    //      ->andReturn( true );
    //  $this->tree
    //      ->shouldReceive( 'to_body' )
    //      ->once()
    //      ->andReturn(
    //          array(
    //              'tree' => array(
    //                  array(
    //                      'path'    => '_posts/2015-10-23-new-post.md',
    //                      'type'    => 'blob',
    //                      'content' => 'Post content 1',
    //                      'mode'    => '100644',
    //                  )
    //              )
    //          )
    //      );
    //  $this->tree
    //      ->shouldReceive( 'set_sha' )
    //      ->once()
    //      ->with( 'c0d2cb90b51826096c826b61a0e74d2c973d7ad8' );
    //  $this->commit
    //      ->shouldReceive( 'to_body' )
    //      ->once()
    //      ->andReturn( array(
    //          'tree'    => 'c0d2cb90b51826096c826b61a0e74d2c973d7ad8',
    //          'message' => 'Commit message',
    //          'parents' => array( 'c0d2cb90b51826096c826b61a0e74d2c973d7ad8' )
    //      ) );
    //  $this->set_post_trees( true );
    //  $this->set_post_commits( true );
    //  $this->set_patch_refs_heads_master( true );

    //  $this->assertTrue( $this->persist->commit( $this->commit ) );
    // }

    // public function test_should_create_authored_commit() {
    //  $this->tree
    //      ->shouldReceive( 'is_changed' )
    //      ->once()
    //      ->andReturn( true );
    //  $this->tree
    //      ->shouldReceive( 'to_body' )
    //      ->once()
    //      ->andReturn(
    //          array(
    //              'tree' => array(
    //                  array(
    //                      'path'    => '_posts/2015-10-23-new-post.md',
    //                      'type'    => 'blob',
    //                      'content' => 'Post content 1',
    //                      'mode'    => '100644',
    //                  )
    //              )
    //          )
    //      );
    //  $this->tree
    //      ->shouldReceive( 'set_sha' )
    //      ->once()
    //      ->with( 'c0d2cb90b51826096c826b61a0e74d2c973d7ad8' );
    //  $this->commit
    //      ->shouldReceive( 'to_body' )
    //      ->once()
    //      ->andReturn( array(
    //          'tree'    => 'c0d2cb90b51826096c826b61a0e74d2c973d7ad8',
    //          'message' => 'Commit message',
    //          'parents' => array( 'c0d2cb90b51826096c826b61a0e74d2c973d7ad8' )
    //      ) );
    //  $this->set_post_trees( true );
    //  update_option( '_wogh_export_user_id', $this->factory->user->create( array(
    //      'display_name' => 'James DiGioia',
    //      'user_email'   => 'jamesorodig@gmail.com',
    //  ) ) );
    //  $this->set_post_commits( true, false );
    //  $this->set_patch_refs_heads_master( true );

    //  $this->assertTrue( $this->persist->commit( $this->commit ) );
    // }
}