Thanks to visit codestin.com
Credit goes to github.com

Skip to content
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Moved CONTRIBUTING.md to project root to conform with github projec…
…t standards.

Updated `CONTRIBUTING.md` with additional info on project setup.
  • Loading branch information
jgebal committed Aug 25, 2017
commit 0d017be44d3a56365571bef3236dc4beb742e819
3 changes: 3 additions & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -12,3 +12,6 @@ release/
*.gz
*.zip
node_modules/
utPLSQL/
utPLSQL-cli/

149 changes: 124 additions & 25 deletions CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -7,31 +7,27 @@ Changes are welcome from all members of the Community.

1. Create a [GitHub Account](https://github.com/join).
2. Fork the utPLSQL Repository and setup your local Repository.
* Each of the steps below are detailed in the [How to Fork](https://help.github.com/articles/fork-a-repo) article!
* Each of the steps below are detailed in the [how to Fork](https://help.github.com/articles/fork-a-repo) article!
* Clone your Fork to your local machine.
* Configure "upstream" remote to the [master utPLSQL repository](https://github.com/utPLSQL/utPLSQL.git).
* Update the git submodules by issuing command: [git submodule update --remote --merge](http://stackoverflow.com/a/21195182)
3. For each change you want to make:
* Create a new branch for your change.
* Configure "upstream" remote to the [utPLSQL repository](https://github.com/utPLSQL/utPLSQL.git).
3. For each change you want to make:
* Make sure your forked repository is up to date with upstream before you start coding. See [syncing your local repository](https://help.github.com/articles/syncing-a-fork) with upstream utPLSQL repository.
* Create a new branch for your change. We use `feature/feature_name` or `bugfix/fixed_issue_name` to identify branch types.
* Make your change in your new branch.
* Although changes can be made in the master branch, it easier long term if a new branch is used.
* Make sure your change is covered with unit tests and/or is represented in examples
* Make sure your change is covered with unit tests.
* **Verify code compiles and all existing and new unit tests pass.**
* The quickest way to have a Pull Request not be accepted, is to submit code that does not compile or pass tests.
* The quickest way to have a Pull Request not approved, is to submit code that does not compile or pass tests.
* Commit change to your local repository.
* Push change to your remote repository
* Submit a [Pull Request](https://help.github.com/articles/using-pull-requests).
* Note: local and remote branches can be deleted after pull request has been accepted.

**Note:** Getting changes from others requires [Syncing your Local repository](https://help.github.com/articles/syncing-a-fork) with Master utPLSQL repository. This can happen at any time.

* Push change to your remote repository.
* Submit a [Pull Request](https://help.github.com/articles/using-pull-requests) into develop branch.
* Note: local and remote branches can be deleted after pull request has been merged.

## Coding Standards ##

* Snake case will be used. This separates keywords in names with underscores. `execute_test`
* All names will be lower case.
* We use snake case for all identifiers in PLSQL code. This separates keywords in names with underscores. `execute_test`
* All code is lower case.
* Prefixes:
* Arguments to procedures and functions will start with `a_` an Example would be procedure `is_valid(a_owner_name varchar2);`
* Arguments to procedures and functions will start with `a_` an Example would be procedure `is_valid(a_owner_name varchar2)`
* Object types and packages will start with `ut_`
* Local variables `l_`
* Global variables `g_`
Expand All @@ -41,27 +37,130 @@ Changes are welcome from all members of the Community.
* varchar2 lengths are set in characters not bytes


## Testing Environment ##
## Configuring local environment ##

Your local environment can be of any flavor (Unix/Linux/Windows/Mac).
At minimum you need to have Oracle database 11.2 XE accessible for the project and SYS account access to install and develop utPLSQL.

We use four different database accounts (users) for development process.
* `ut3_latest_release` - holds latest released version of utPLSQL. This schema holds the testing framework used for self-testing of utPLSQL development.
* `ut3` - holds latest (current) development version of utPLSQL. This is the schema you will be working on.
* `ut3_tester` - holds unit test packages for development of utPLSQL.
* `ut3$user#` - used for testing accessibility to schema names with special characters.



Snippet to get you started with development.

_If you're using Windows, you can run below scripts using `GIT bash` - Windows-based Unix-like command line._

```bash
# clone your fork of utPLSQL
git clone https://github.com/your account/utPLSQL.git utPLSQL

cd utPLSQL

# add main porject repo as upstream
git remote add upstream https://github.com/utPLSQL/utPLSQL.git

# fetch all remote repositories
git fetch --all

# clone utPLSQL master branch from upstream into utPLSQL sub-directory of your project
git clone --depth=1 --branch=master https://github.com/utPLSQL/utPLSQL.git

# download beta version of utPLSQL-cli
curl -Lk -o utPLSQL-cli.zip https://bintray.com/viniciusam/utPLSQL-cli/download_file?file_path=utPLSQL-cli-develop-test3.zip
# unzip utPLSQL-cli and remove the zip file
unzip utPLSQL-cli.zip && chmod -R u+x utPLSQL-cli && rm utPLSQL-cli.zip

```

Refreshing your local repo.
```bash
# fetch all remote repositories
git fetch --all

# remove sub-direcotry containing master branch shallow copy
rm -rf utPLSQL/*
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

rm -rf utPLSQL

Otherwise it will lead to an error with following clone command

# clone utPLSQL master branch from upstream into utPLSQL sub-directory of your project
git clone --depth=1 --branch=master https://github.com/utPLSQL/utPLSQL.git

rm -rf utPLSQL-cli/*
# download beta version of utPLSQL-cli
curl -Lk -o utPLSQL-cli.zip https://bintray.com/viniciusam/utPLSQL-cli/download_file?file_path=utPLSQL-cli-develop-test3.zip
# unzip utPLSQL-cli and remove the zip file
unzip utPLSQL-cli.zip && chmod -R u+x utPLSQL-cli && rm utPLSQL-cli.zip

```

Cleanup of utPLSQL installation.
```sql
drop user ut3 cascade;
drop user ut3_latest_release cascade;
drop user ut3_tester cascade;
drop user ut3$user# cascade;

begin
for i in (
select decode(owner,'PUBLIC','drop public synonym "','drop synonym "'||owner||'"."')|| synonym_name ||'"' drop_orphaned_synonym from dba_synonyms a
where not exists (select null from dba_objects b where a.table_name=b.object_name and a.table_owner=b.owner )
) loop
execute immediate i.drop_orphaned_synonym;
end loop;
end;
/
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Doesn't work inside 12c PDBs, raises ORA-65040: operation not allowed from within a pluggable database

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

any alternative or any idea how to get this to work?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry I didn't come with a suggestion in the first place.
Problem is that it tries to drop SYS-Synonyms, which is not allowed for PDBS.
The following should do it:

begin
  for i in (
    select decode(owner,'PUBLIC','drop public synonym "','drop synonym "'||owner||'"."')|| synonym_name ||'"' drop_orphaned_synonym from dba_synonyms a
     where a.table_owner <> 'SYS' and not exists (select null from dba_objects b where a.table_name=b.object_name and a.table_owner=b.owner )
  ) loop
    execute immediate i.drop_orphaned_synonym;
  end loop;
end;

```

We are using docker images to test utPLSQL on our Travis CI builds. The following versions of Oracle Database are being used.
Install utPLSQL for development
```bash
export UT3_OWNER=ut3
export UT3_OWNER_PASSWORD=ut3
export UT3_RELEASE_VERSION_SCHEMA=ut3_latest_release
export UT3_TESTER=ut3_tester
export UT3_TESTER_PASSWORD=ut3
export UT3_TABLESPACE=users
export UT3_USER="UT3\$USER#"
export UT3_USER_PASSWORD=ut3
export SQLCLI=sql
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

shouldn't that be

SQLCI=sqlplus

?

export CONNECTION_STR=127.0.0.1:1521/orcl
export ORACLE_PWD=oracle

.travis/install.sh
.travis/install_utplsql_release.sh
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

$ .travis/install_utplsql_release.sh

cd $UTPLSQL_DIR/source
.travis/install_utplsql_release.sh: line 5: cd: /source: No such file or directory

I guess exporting UTPLSQL_DIR is missing

.travis/create_additional_grants_for_old_tests.sh
```

Reinstalling utPLSQL development in `ut3` schema.
```bash
cd source
sqlplus sys/oracle@orcl as sysdba <<-SQL
@uninstall ut3
@install ut3
SQL
```

## Build Environment ##

We are using private docker images to test utPLSQL for our Travis CI builds. The following versions of Oracle Database are being used.

* 11g XE R2
* 12c SE R1
* 12c SE R2

These images are based on the official dockerfiles released by Oracle, but due to licensing restrictions, we can't make the images public. You can build your own and use it locally, or push to a private docker repository.
These images are based on the slimmed versions [official dockerfiles released by Oracle](https://github.com/utPLSQL/docker-scripts), but due to licensing restrictions, we can't make the images public.
You can build your own and use it locally, or push to a private docker repository.

The build steps are simple if you already have some experience using Docker. You can find detailed information about how to build your own image with a running database in: [example of creating an image with pre-built DB](https://github.com/oracle/docker-images/blob/master/OracleDatabase/samples/prebuiltdb/README.md)

> You can find more info about the official Oracle images on the [Oracle Database on Docker](https://github.com/oracle/docker-images/tree/master/OracleDatabase) GitHub page.

> If you are new to Docker, you can start by reading the [Getting Started With Docker](https://docs.docker.com/engine/getstarted/) docs.

### Build Notes ###
* You need to comment out the VOLUME line. This step is required, because volumes are not saved when using `docker commit` command.

* When the build proccess is complete, you will run the container to install the database. Once everything is set up and you see the message "DATABASE IS READY!", you may change the password and stop the running container. After the container is stopped, you can safely commit the container.
### Docker Build Notes ###

* You need to comment out the VOLUME line. This step is required, because volumes are not saved when using `docker commit` command.
* When the build process is complete, you will run the container to install the database. Once everything is set up and you see the message "DATABASE IS READY!", you may change the password and stop the running container. After the container is stopped, you can safely commit the container.
* You can use the --squash experimental docker tag to reduce the image size. Example:
```
docker build --force-rm --no-cache --squash -t oracle/db-prebuilt .
Expand All @@ -75,7 +174,7 @@ Variable | Description

### SQLCL ###

Our build configurarion uses SQLCL to run the scripts, and you need to configure a few additional secure environment variables. After the first build, the downloaded file will be cached.
Our build configuration uses SQLCL to run the scripts, and you need to configure a few additional secure environment variables. After the first build, the downloaded file will be cached.

Variable | Description
---------|------------
Expand Down