Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[CBRD-24519] Add test case for PL/CSQL test cases 09 (feature/plcsql-p1) #1769

Merged
merged 13 commits into from
Jul 19, 2024

Conversation

swi0110
Copy link
Contributor

@swi0110 swi0110 commented Jul 15, 2024

@swi0110 swi0110 requested review from kwonhoil and ssihil as code owners July 15, 2024 09:26
@swi0110 swi0110 changed the title [CBRD-24519] Add tets case for PL/CSQL test cases 09 (feature/plcsql-p1) [CBRD-24519] Add test case for PL/CSQL test cases 09 (feature/plcsql-p1) Jul 15, 2024
@swi0110 swi0110 requested a review from hgryoo July 15, 2024 09:27
@swi0110 swi0110 requested a review from tw-kang as a code owner July 16, 2024 03:32
@swi0110 swi0110 removed the request for review from tw-kang July 16, 2024 04:21
@swi0110 swi0110 closed this Jul 16, 2024
@swi0110 swi0110 deleted the cbrd_25420_plcsql-p1_09 branch July 16, 2024 05:25
@swi0110 swi0110 reopened this Jul 16, 2024
@swi0110 swi0110 requested a review from kwonhoil July 18, 2024 04:23
@swi0110 swi0110 merged commit 6411f60 into CUBRID:feature/plcsql-p1 Jul 19, 2024
hgryoo added a commit that referenced this pull request Dec 12, 2024
…op (#2000)

http://jira.cubrid.org/browse/CBRD-25184

## 1. PL/CSQL Enhancements

### 1.1 New Test Cases
- **[CBRD-24519]** Added test cases for PL/CSQL (Cases 10–23).
  - **Pull Requests**: #1769, #1779, #1894, #1903, #1907, #1912, #1918, #1922, #1924, #1925
  - **Details**:
    - Covered scenarios involving OUT parameters, runtime exceptions, and keyword-built-in function overlaps (e.g., `CURRENT_USER`).
    - Validated stored procedure behavior with user schema prefixes.

- **[CBRD-25261]** Modified answers for PL/CSQL tests.
  - **Pull Request**: #1761
  - Adjusted for grammar refinements and future merges.

### 1.2 System Changes
- **[CBRD-25184]** Revised answers due to grammar changes.
  - **Pull Requests**: #1770, #1878, #2001
  - **Changes**:
    - Updates for keywords (e.g., `REVERSE`), user schema handling, and new system catalog columns.

---

## 2. Bug Fixes

### 2.1 Error Handling
- **[CBRD-25303]** Fixed issues in ADDTIME() with invalid XASL content.
  - **Pull Request**: #1973

- **[CBRD-25273]** Modified test case for ADDDATE() returning NULL.
  - **Pull Request**: #1974

### 2.2 Typographical Fixes
- **[CBRD-24707]** Resolved large-scale test failures due to typos.
  - **Pull Request**: #1897

---

## 3. Test Improvements for Errors

### 3.1 VARCHAR Conversion
- **[CBRD-25345]** Added SQL test for errors converting VARCHAR to TIME/TIMESTAMP.
  - **Pull Request**: #1975

### 3.2 Error Messaging
- **[CBRD-25673]** Improved error message consistency in JDBC.
  - **Pull Request**: #1958

- **[CBRD-25416]** Resolved stored procedure syntax display issues.
  - **Pull Requests**: #1963, #1967

---

## 4. Stored Procedure System Updates

### 4.1 Schema Updates
- **[CBRD-25352]** Introduced user schema for stored procedures.
  - **Pull Request**: #1824

### 4.2 Authorization Handling
- **[CBRD-24786]** Updated answers for `SHOW GRANTS` and `db_auth` behavior.
  - **Pull Requests**: #1823, #1802

## Contributors Overview (excluding merge commits)
- **Song Won-ryong** (@swi0110) - 28 commits
- **Hyung-Gyu Ryoo** (@hgryoo) - 26 commits
- **junsklee** (@junsklee) - 7 commits
- **jongmin-won** (@jongmin-won) - 6 commit
- **zionyun** (@zionyun) - 1 commit
- **kwonhoil** (@kwonhoil) - 1 commit
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants