aboutsummaryrefslogtreecommitdiffstatshomepage
path: root/OpenSim/Region/CoreModules/Avatar/Attachments/Tests (follow)
Commit message (Collapse)AuthorAgeFilesLines
* Add regression TestRezScriptedAttachmentsFromInventory() though this ↵Justin Clark-Casey (justincc)2012-07-111-1/+26
| | | | currently only checks for the presence of script items, not for started scripts
* Do not allow a script to attach a prim if its being sat upon.Justin Clark-Casey (justincc)2012-07-091-1/+31
| | | | | | This prevents a stack overflow where a get position on the avatar will refer to the attachment which will in turn refer back to the avatar. This required recording of all sitting avatars on a prim which is done separately from recording the sit target avatar. Recording HashSet is null if there are no sitting avatars in order to save memory.
* minor: Remove some wrong comments in attachments regression testsJustin Clark-Casey (justincc)2012-07-061-2/+0
|
* Add assert to attachment regression tests to check that number of objects in ↵Justin Clark-Casey (justincc)2012-07-061-0/+14
| | | | the scene graph
* Add OSSL function osForceAttachToAvatarFromInventory()Justin Clark-Casey (justincc)2012-07-051-1/+1
| | | | | | | This works like osForceAttachToAvatar() but allows an object to be directly specified from the script object's inventory rather than forcing it to be rezzed in the scene first. Still only attaches objects to the owner of the script. This allows one to bypass the complicated co-ordination of first rezzing objects in the scene before attaching them. Threat level high.
* Change AttachmentsModule.DetachSingleAttachmentToInv() to accept a SOG ↵Justin Clark-Casey (justincc)2012-06-281-3/+4
| | | | | | | directly instead of an item ID to then shuffle through attachments, saving CPU busywork. Almost all callers already had the sog to hand. Still checking that it's really an attachment, but now by inspecting SOG.AttachedAvatar
* Automatically disable log4net before each regression test so that logging is ↵Justin Clark-Casey (justincc)2012-06-271-3/+1
| | | | | | | confined to a single test if it's turned on. This involves making test classes inherit from a common OpenSimTestCase. This will be applied to more classes as required.
* refactor: Remove unnecessary AttachmentModuleTests.m_userId in favour of ↵Justin Clark-Casey (justincc)2012-06-261-40/+26
| | | | local variables
* refactor: Use local attachment module variables instead of global m_attMod.Justin Clark-Casey (justincc)2012-06-261-11/+10
| | | | This also avoids confusion between tests where one sets up m_attMod and another accidentally uses it after failing to set one up itself.
* refactor: make m_presence a local variable in all AttachmentsModuleTests ↵Justin Clark-Casey (justincc)2012-06-261-26/+25
| | | | since it doesn't need to be global and some tests set up more than one sp
* Add regression test to check that attachments in source region are deleting ↵Justin Clark-Casey (justincc)2012-06-261-18/+115
| | | | when an agent teleports to a neighbouring region
* refactor AttachmentsModule tests to use a common method for standard ↵Justin Clark-Casey (justincc)2012-06-221-70/+69
| | | | attachment item setup
* Add regression test for updating attachment positionJustin Clark-Casey (justincc)2012-06-221-0/+30
|
* Fix bug where an avatar that had an object they owned attached through ↵Justin Clark-Casey (justincc)2012-05-231-2/+11
| | | | | | | llAttachToAvatar() or osForceAttachToAvatar() would wrongly have next permissions come into play when they detached that object and rezzed it in scene. This is because the attachments module code was setting the 'object slam' bit by using PermissionMask.All Solution here is to route the attachment item creation call through the existing inventory code in BasicInventoryAccessModule rather than copy/pasted code in AttachmentsModule itself.
* Add regression test for teleporting an agent between separated regions on ↵Justin Clark-Casey (justincc)2012-04-271-1/+1
| | | | | | | | the same simulator. This involves a large amount of change in test scene setup code to allow test scenes to share shared modules SetupScene is now an instance method that requires an instantiation of SceneHelpers, though other SceneHelpers methods are still static May split these out into separate classes in the future.
* Store FromItemID for attachments once on SOG instead of on every SOP and ↵Justin Clark-Casey (justincc)2012-04-071-2/+3
| | | | | | only ever using the root part entry. This eliminates some pointless memory use.
* Removed all refs to IClientAPI from IAttachmentsModule. Separated client ↵Dan Lake2011-10-041-1/+1
| | | | handlers for attachments to call public interface and rearranged module file into sections
* Removed redundant code in AttachmentsModule and simplified interfaces which ↵Dan Lake2011-10-031-5/+5
| | | | converted back and forth between ScenePresence and IClientAPI. More to be done still.
* Don't try and delete attachments for child agent closeJustin Clark-Casey (justincc)2011-09-131-0/+32
|
* Remember to set and unset the fire and forget method at the top of the ↵Justin Clark-Casey (justincc)2011-09-081-3/+7
| | | | attachment and npc tests
* In SetAttachment, if the existing attachment has no asset id then carry on ↵Justin Clark-Casey (justincc)2011-09-061-0/+8
| | | | | | | | | | | rather than abort. When a user logs in, the attachment item ids are pulled from persistence in the Avatars table. However, the asset ids are not saved. When the avatar enters a simulator the attachments are set again. If we simply perform an item check then the asset ids (which are now present) are never set, and NPC attachments later fail unless the attachment is detached and reattached. Hopefully resolves part of http://opensimulator.org/mantis/view.php?id=5653
* Stop the pointless double setting of every attachment in AvatarAppearance.Justin Clark-Casey (justincc)2011-09-061-0/+1
| | | | The second was already being filtered out so this has no user level effect
* Make SP.Attachments available as sp.GetAttachments() instead.Justin Clark-Casey (justincc)2011-08-311-5/+5
| | | | | | | The approach here, as in other parts of OpenSim, is to return a copy of the list rather than the attachments list itself This prevents callers from forgetting to lock the list when they read it, as was happening in various parts of the codebase. It also improves liveness. This might improve attachment anomolies when performing region crossings.
* refactor: migrate DropObject handling fully into AttachmentsModule from SceneJustin Clark-Casey (justincc)2011-08-301-5/+2
|
* Fix bug where attachments were remaining on the avatar after being dropped.Justin Clark-Casey (justincc)2011-08-301-4/+10
| | | | If the inventory service is configured not to allow deletion then these will not disappear from inventory
* Eliminate duplicate AttachmentPoint properties by always using the one ↵Justin Clark-Casey (justincc)2011-08-271-3/+0
| | | | stored in the root part's state field.
* refactor: move SOP.IsAttachment and AttachmentPoint up into SOG to avoid ↵Justin Clark-Casey (justincc)2011-08-271-9/+9
| | | | pointless duplication of identical values
* refactor: remove common presence set up in attachments testsJustin Clark-Casey (justincc)2011-08-261-32/+38
|
* comment out verbose test logging from last commitJustin Clark-Casey (justincc)2011-08-261-1/+1
|
* add regression test for detaching an attachment to the sceneJustin Clark-Casey (justincc)2011-08-261-0/+33
|
* refactor: move sog.DetachToInventoryPrep() into ↵Justin Clark-Casey (justincc)2011-08-261-1/+1
| | | | AttachmentsModule.DetachSingleAttachmentToInv()
* Remove forcing of phantom on ground attached objects - attachments can be ↵Justin Clark-Casey (justincc)2011-08-241-3/+0
| | | | | | both non-phantom and flagged as physical. As per Melanie
* extend initial rez regression test to check that attachment is phantomJustin Clark-Casey (justincc)2011-08-241-9/+14
|
* Make objects attached from the ground phantomJustin Clark-Casey (justincc)2011-08-241-2/+10
|
* add TestAddAttachmentFromGround() regression testJustin Clark-Casey (justincc)2011-08-241-1/+31
|
* rename AttachmentsModule.ShowDetachInUserInventory() to ↵Justin Clark-Casey (justincc)2011-08-231-1/+1
| | | | DetachSingleAttachmentToInv() for consistency and to reflect it's actual behaviour
* remove mono compiler warningsJustin Clark-Casey (justincc)2011-08-231-2/+0
|
* replace TestRemoveAttachments() with a more thorough TestRemoveAttachment()Justin Clark-Casey (justincc)2011-08-231-51/+27
|
* replace old TestAddAttachments() with a more thorough TestAddAttachment()Justin Clark-Casey (justincc)2011-08-231-7/+28
|
* extend test to check that there is one attachment and that it has the right nameJustin Clark-Casey (justincc)2011-08-171-1/+6
|
* Add new regression TestRezAttachmentsOnAvatarEntrance() to do simple ↵Justin Clark-Casey (justincc)2011-08-171-6/+16
| | | | attachments check
* Add new FireAndForgetMethod.None.Justin Clark-Casey (justincc)2011-08-161-13/+33
| | | | This executes the callback on the same thread that made the request. Designed for use only by regression tests that rely on a predicable event ordering.
* Move some previously common code back into separate tests. Remove unused ↵Justin Clark-Casey (justincc)2011-08-161-10/+9
| | | | region handle from test.
* Isolate existing incomplete attachments tests rather than have them rely on ↵Justin Clark-Casey (justincc)2011-08-161-5/+5
| | | | | | each other. Much easier to debug this way.
* drop number of attachments in test from 3 to 2 to reduce text complexityJustin Clark-Casey (justincc)2011-08-161-4/+1
|
* remove setting up of second scene in attachments since it's not currently usedJustin Clark-Casey (justincc)2011-08-161-24/+21
|
* relocate AttachmentTests.cs to AttachmentsModuleTests.csJustin Clark-Casey (justincc)2011-08-161-0/+172