diff --git a/.github/workflows/oci.yml b/.github/workflows/oci.yml new file mode 100644 index 00000000000..2809a64ede5 --- /dev/null +++ b/.github/workflows/oci.yml @@ -0,0 +1,74 @@ +name: PHPUnit + +on: + pull_request: + push: + branches: + - master + - stable* + +env: + APP_NAME: text + +jobs: + oci: + runs-on: ubuntu-latest + + strategy: + # do not stop on another job's failure + fail-fast: false + matrix: + php-versions: ['7.4'] + databases: ['oci'] + server-versions: ['master'] + + name: php${{ matrix.php-versions }}-${{ matrix.databases }}-${{ matrix.server-versions }} + + services: + oracle: + image: deepdiver/docker-oracle-xe-11g # "wnameless/oracle-xe-11g-r2" + ports: + - "1521:1521" + + steps: + - name: Checkout server + uses: actions/checkout@v2 + with: + repository: nextcloud/server + ref: ${{ matrix.server-versions }} + + - name: Checkout submodules + shell: bash + run: | + auth_header="$(git config --local --get http.https://github.com/.extraheader)" + git submodule sync --recursive + git -c "http.extraheader=$auth_header" -c protocol.version=2 submodule update --init --force --recursive --depth=1 + + - name: Checkout app + uses: actions/checkout@v2 + with: + path: apps/${{ env.APP_NAME }} + + - name: Set up PHPUnit + working-directory: apps/${{ env.APP_NAME }} + run: composer i + + - name: Set up php ${{ matrix.php-versions }} + uses: "shivammathur/setup-php@v2" + with: + php-version: "${{ matrix.php-versions }}" + extensions: mbstring, iconv, fileinfo, intl, oci8 + tools: phpunit:8.5.2 + coverage: none + + - name: Set up Nextcloud + run: | + mkdir data + ./occ maintenance:install --verbose --database=oci --database-name=XE --database-host=127.0.0.1 --database-port=1521 --database-user=autotest --database-pass=owncloud --admin-user admin --admin-pass admin + php -f index.php + ./occ app:enable --force ${{ env.APP_NAME }} + + - name: PHPUnit + working-directory: apps/${{ env.APP_NAME }}/tests + run: phpunit -c phpunit.xml + diff --git a/lib/Migration/Version030201Date20201116110353.php b/lib/Migration/Version030201Date20201116110353.php new file mode 100644 index 00000000000..ab7ffa5fa7d --- /dev/null +++ b/lib/Migration/Version030201Date20201116110353.php @@ -0,0 +1,60 @@ +isOracle = $config->getSystemValue('dbtype', 'sqlite') === 'oci'; + } + + public function changeSchema(IOutput $output, Closure $schemaClosure, array $options) { + /** @var ISchemaWrapper $schema */ + $schema = $schemaClosure(); + + $this->ensureColumnIsNullable($schema, 'text_steps', 'version'); + + if ($this->isOracle) { + // Drop table if we are on oracle and recreate it with the next migration Version030201Date20201116123153 + if ($schema->hasTable('text_documents')) { + $schema->dropTable('text_documents'); + } + } else { + $this->ensureColumnIsNullable($schema, 'text_documents', 'current_version'); + $this->ensureColumnIsNullable($schema, 'text_documents', 'last_saved_version'); + $table = $schema->getTable('text_documents'); + $column = $table->getColumn('id'); + if ($column->getAutoincrement()) { + $table->changeColumn('id', [ + 'autoincrement' => false, + ]); + } + } + + return $schema; + } + + protected function ensureColumnIsNullable(ISchemaWrapper $schema, string $tableName, string $columnName): bool { + $table = $schema->getTable($tableName); + $column = $table->getColumn($columnName); + + if ($column->getNotnull()) { + $column->setNotnull(false); + return true; + } + + return false; + } +} diff --git a/lib/Migration/Version030201Date20201116123153.php b/lib/Migration/Version030201Date20201116123153.php new file mode 100644 index 00000000000..abacaa6fad2 --- /dev/null +++ b/lib/Migration/Version030201Date20201116123153.php @@ -0,0 +1,69 @@ +isOracle = $config->getSystemValue('dbtype', 'sqlite') === 'oci'; + } + + public function changeSchema(IOutput $output, Closure $schemaClosure, array $options): ?ISchemaWrapper { + if (!$this->isOracle) { + return null; + } + + /** @var ISchemaWrapper $schema */ + $schema = $schemaClosure(); + + if (!$schema->hasTable('text_documents')) { + // Recreate table from the first migration since we cannot alter the autoincrement on the id column with oracle + $table = $schema->createTable('text_documents'); + $table->addColumn('id', Type::BIGINT, [ + 'notnull' => true, + 'unsigned' => true, + ]); + $table->addColumn('current_version', Type::BIGINT, [ + // 'notnull' => true, + 'notnull' => false, + 'default' => 0, + 'unsigned' => true, + ]); + $table->addColumn('last_saved_version', Type::BIGINT, [ + // 'notnull' => true, + 'notnull' => false, + 'default' => 0, + 'unsigned' => true, + ]); + $table->addColumn('last_saved_version_time', Type::BIGINT, [ + 'length' => 20, + 'unsigned' => true, + ]); + $table->addColumn('last_saved_version_etag', Type::STRING, [ + 'notnull' => false, + 'length' => 64, + 'default' => '' + ]); + $table->addColumn('base_version_etag', Type::STRING, [ + 'notnull' => false, + 'length' => 64, + 'default' => '' + ]); + $table->setPrimaryKey(['id']); + return $schema; + } + return null; + } +}