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

DATAREDIS-549 - Migrate tests to AssertJ. #225

Closed
wants to merge 20 commits into from

Conversation

Projects
None yet
5 participants
@mp911de
Copy link
Member

commented Oct 21, 2016

Migrate tests to AssertJ.


Related issue: DATAREDIS-549

odrotbohm and others added some commits Nov 23, 2016

DATAREDIS-552 - Support client name configuration for JedisConnection…
…Factory.

Motivation:
JedisConnectionFactory should support setting clientName for Jedis connections
so to improve troubleshooting.

Result:
Easier to troubleshooting with connection name.

Original pull request: #219.
DATAREDIS-552 - Polishing.
Apply client name to Sentinel and unpooled connections if set. Add ticket references to JavaDoc. Add tests. Fix existing JavaDoc letter casing and punctuation. Rearrange isClusterAware/isRedisSentinelAware methods.

Original pull request: #219.
DATAREDIS-579 - Register RedisRepositoryFactory in spring.factories.
This is required for the switch in support for multi-store detection.

Related ticket: DATACMNS-952.
DATAREDIS-532 - Enhance JavaDoc and convert links to Redis documentat…
…ion.

Extend JavaDoc documentation for Command/Connection/Operation interfaces. Align method ordering in operation interfaces with connection/commands interfaces.
DATAREDIS-589 - Move secondary index cleanup to MappingExpirationList…
…ener.

We now no longer rely on ApplicationEvents captured in the RedisKeyValueAdapter for performing cleanup operations for expired keys, but do this along with the phantom key removal. This removes a flaw when initializing a non repository related KeyspaceEventListener publishing events that actually are unrelated to the Adapter.

Additionally upgraded test infrastructure to utilize Redis 3.2.6 with disabled protected-mode and enabled keyspace-events.

Original pull request: #232.

@mp911de mp911de changed the title DATAREDIS-549, DATAREDIS-563 - Migrate Jira ticket reference format in tests and tests to AssertJ. DATAREDIS-549 - Migrate tests to AssertJ. Jan 12, 2017

DATAREDIS-589 - Polishing.
Run expiry tests only when executing long-running tests. Guard RedisConnectionFactory against multiple initializations. Synchronize key expiry. Remove unused code. Extend years in license header.

Original pull request: #232.

@mp911de mp911de force-pushed the master branch from c9a4cd4 to d00ab8c Jan 12, 2017

@mp911de mp911de force-pushed the issue/DATAREDIS-549 branch from 0d196ce to bf7f746 Jan 12, 2017

@mp911de mp911de force-pushed the master branch from 24240f8 to 88bd5e3 Mar 30, 2017

@mp911de mp911de force-pushed the master branch from 0f9d0a9 to 0955a4f May 2, 2018

spring-operator added some commits Mar 18, 2019

DATAREDIS-948 - URL Cleanup.
This commit updates URLs to prefer the https protocol. Redirects are not followed to avoid accidentally expanding intentionally shortened URLs (i.e. if using a URL shortener).

# Fixed URLs

## Fixed Success
These URLs were switched to an https URL with a 2xx status. While the status was successful, your review is still recommended.

* http://maven.apache.org/xsd/maven-4.0.0.xsd with 1 occurrences migrated to:
  https://maven.apache.org/xsd/maven-4.0.0.xsd ([https](https://maven.apache.org/xsd/maven-4.0.0.xsd) result 200).

# Ignored
These URLs were intentionally ignored.

* http://maven.apache.org/POM/4.0.0 with 2 occurrences
* http://www.w3.org/2001/XMLSchema-instance with 1 occurrences

Original pull request: #403
DATAREDIS-948 - URL Cleanup.
This commit updates URLs to prefer the https protocol. Redirects are not followed to avoid accidentally expanding intentionally shortened URLs (i.e. if using a URL shortener).

# Fixed URLs

## Fixed Success
These URLs were switched to an https URL with a 2xx status. While the status was successful, your review is still recommended.

* [ ] http://www.apache.org/licenses/ with 1 occurrences migrated to:
  https://www.apache.org/licenses/ ([https](https://www.apache.org/licenses/) result 200).
* [ ] http://www.apache.org/licenses/LICENSE-2.0 with 501 occurrences migrated to:
  https://www.apache.org/licenses/LICENSE-2.0 ([https](https://www.apache.org/licenses/LICENSE-2.0) result 200).

Original Pull Request: #430
@mp911de

This comment has been minimized.

Copy link
Member Author

commented Jun 28, 2019

Closing. Not going to happen with this PR.

@mp911de mp911de closed this Jun 28, 2019

@mp911de mp911de deleted the issue/DATAREDIS-549 branch Jun 28, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.